Fire create_upper_paths_hook for UPPERREL_PARTIAL_GROUP_AGG

create_partial_grouping_paths() called the FDW callback
GetForeignUpperPaths for UPPERREL_PARTIAL_GROUP_AGG, but never the
general-purpose create_upper_paths_hook that non-FDW extensions rely
on.  create_partial_distinct_paths() already fires both for
UPPERREL_PARTIAL_DISTINCT.

Add the missing call right next to GetForeignUpperPaths, so an
extension can add partial-aggregate paths here too, before the caller
gathers them and builds a Finalize Aggregate on top.

Reviewed-by: Bryan Green <[email protected]>
Reviewed-by: Himanshu Upadhyaya <[email protected]>
Reviewed-by: Shihao Zhong <[email protected]>
Discussion: 
https://postgr.es/m/[email protected]

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/e8a82ddeaf69b077125cf9303df809234461bd37

Modified Files
--------------
src/backend/optimizer/plan/planner.c | 6 ++++++
1 file changed, 6 insertions(+)

Reply via email to