aboutsummaryrefslogtreecommitdiff
path: root/src/backend/executor/nodeAgg.c
diff options
context:
space:
mode:
authorDavid Rowley <drowley@postgresql.org>2025-02-19 00:42:22 +1300
committerDavid Rowley <drowley@postgresql.org>2025-02-19 00:42:22 +1300
commit593509202f669dbc4a9db33bb3aca2bd68f7ab5c (patch)
tree0a5b9508b209bf30c907b42d7948bd89c6446af5 /src/backend/executor/nodeAgg.c
parent217919dd0954f54402e8d0a38cd203a740754077 (diff)
downloadpostgresql-593509202f669dbc4a9db33bb3aca2bd68f7ab5c.tar.gz
postgresql-593509202f669dbc4a9db33bb3aca2bd68f7ab5c.zip
Fix poorly written regression test
bd10ec529 added code to allow redundant functionally dependent GROUP BY columns to be removed using unique indexes and NOT NULL constraints as proofs of functional dependency. In that commit, I (David) added a test to ensure that when there are multiple indexes available to remove columns that we pick the index that allows us to remove the most columns. This test was faulty as it assumed the t3 table's primary key index was valid to use as functional dependency proof, but that's not the case since that's defined as deferrable. Here we adjust the tests added by that commit to use the t2 table instead. That's defined with a non-deferrable primary key. Author: songjinzhou <tsinghualucky912@foxmail.com> Author: David Rowley <dgrowleyml@gmail.com> Reviewed-by: Japin Li <japinli@hotmail.com> Discussion: https://postgr.es/m/tencent_CD414C79D39668455DF80D35143B87634C08@qq.com
Diffstat (limited to 'src/backend/executor/nodeAgg.c')
0 files changed, 0 insertions, 0 deletions