]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Repair logic for reordering grouping sets optimization.
authorAndrew Gierth <rhodiumtoad@postgresql.org>
Sun, 30 Jun 2019 22:49:23 +0000 (23:49 +0100)
committerAndrew Gierth <rhodiumtoad@postgresql.org>
Sun, 30 Jun 2019 22:49:23 +0000 (23:49 +0100)
commit05dc5f4767e1c5ed157b2870f05d57f3378302f4
treef26fd5ab91cb91bdde62f172d07539237f6d112c
parent43085a4f693e35d674a9965c484e4e87f18f6d29
Repair logic for reordering grouping sets optimization.

The logic in reorder_grouping_sets to order grouping set elements to
match a pre-specified sort ordering was defective, resulting in
unnecessary sort nodes (though the query output would still be
correct). Repair, simplifying the code a little, and add a test.

Per report from Richard Guo, though I didn't use their patch. Original
bug seems to have been my fault.

Backpatch back to 9.5 where grouping sets were introduced.

Discussion: https://postgr.es/m/CAN_9JTzyjGcUjiBHxLsgqfk7PkdLGXiM=pwM+=ph2LsWw0WO1A@mail.gmail.com
src/backend/optimizer/plan/planner.c
src/test/regress/expected/groupingsets.out
src/test/regress/sql/groupingsets.sql