]> granicus.if.org Git - postgresql/commit
Improve handling of group-column indexes in GroupingSetsPath.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 9 Mar 2016 03:32:03 +0000 (22:32 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 9 Mar 2016 03:32:11 +0000 (22:32 -0500)
commit9e8b99420fe5f80495ada8dc50aeb7b954b33093
tree0ff90d699a96823ea6c425cb3a6bb6da1fca2143
parenta40814d7aacbda9266c703cc011d3506668899de
Improve handling of group-column indexes in GroupingSetsPath.

Instead of having planner.c compute a groupColIdx array and store it in
GroupingSetsPaths, make create_groupingsets_plan() find the grouping
columns by searching in the child plan node's tlist.  Although that's
probably a bit slower for create_groupingsets_plan(), it's more like
the way every other plan node type does this, and it provides positive
confirmation that we know which child output columns we're supposed to be
grouping on.  (Indeed, looking at this now, I'm not at all sure that it
wasn't broken before, because create_groupingsets_plan() isn't demanding
an exact tlist match from its child node.)  Also, this allows substantial
simplification in planner.c, because it no longer needs to compute the
groupColIdx array at all; no other cases were using it.

I'd intended to put off this refactoring until later (like 9.7), but
in view of the likely bug fix and the need to rationalize planner.c's
tlist handling so we can do something sane with Konstantin Knizhnik's
function-evaluation-postponement patch, I think it can't wait.
src/backend/nodes/outfuncs.c
src/backend/optimizer/plan/createplan.c
src/backend/optimizer/plan/planner.c
src/backend/optimizer/util/pathnode.c
src/backend/optimizer/util/tlist.c
src/include/nodes/relation.h
src/include/optimizer/pathnode.h
src/include/optimizer/tlist.h