Pass additional arguments to a couple of grouping-related functions.
authorRobert Haas <rhaas@postgresql.org>
Thu, 15 Mar 2018 15:33:52 +0000 (11:33 -0400)
committerRobert Haas <rhaas@postgresql.org>
Thu, 15 Mar 2018 15:33:52 +0000 (11:33 -0400)
commit648a6c7bd815f98b35709bd56f9f1ca276b33ae6
treeb858d03e0d2a6eee079f361a575d10ed3a61f25f
parent8d1b805fcc57be47d3269a13eb01d993fe768295
Pass additional arguments to a couple of grouping-related functions.

get_number_of_groups() and make_partial_grouping_target() currently
fish information directly out of the PlannerInfo; in the former case,
the target list, and in the latter case, the HAVING qual.  This works
fine if there's only one grouping relation, but if the pending patch
for partition-wise aggregate gets committed, we'll have multiple
grouping relations and must therefore use appropriately translated
versions of these values for each one.  To make that simpler, pass the
values to be used as arguments.

Jeevan Chalke.  The larger patch series of which this patch is a part
was also reviewed and tested by Antonin Houska, Rajkumar Raghuwanshi,
David Rowley, Dilip Kumar, Konstantin Knizhnik, Pascal Legrand, Rafia
Sabih, and me.

Discussion: http://postgr.es/m/CAM2+6=UqFnFUypOvLdm5TgC+2M=-E0Q7_LOh0VDFFzmk2BBPzQ@mail.gmail.com
Discussion: http://postgr.es/m/CAM2+6=W+L=C4yBqMrgrfTfNtbtmr4T53-hZhwbA2kvbZ9VMrrw@mail.gmail.com
src/backend/optimizer/plan/planner.c