diff options
author | Robert Haas <rhaas@postgresql.org> | 2018-02-27 13:22:36 -0500 |
---|---|---|
committer | Robert Haas <rhaas@postgresql.org> | 2018-02-27 13:23:50 -0500 |
commit | 5e6a63c0d1028b9950c9cbcd7aaf9f2a67880a8d (patch) | |
tree | cc5addc2bbfe6ea165cf7bee77c3478f29efd6bb | |
parent | 3bfe957761ace9290072690a6298fc0eb996d857 (diff) | |
download | postgresql-5e6a63c0d1028b9950c9cbcd7aaf9f2a67880a8d.tar.gz postgresql-5e6a63c0d1028b9950c9cbcd7aaf9f2a67880a8d.zip |
Minor cleanup of code related to partially_grouped_rel.
Jeevan Chalke
Discussion: http://postgr.es/m/CAM2+6=X9kxQoL2ZqZ00E6asBt9z+rfyWbOmhXJ0+8fPAyMZ9Jg@mail.gmail.com
-rw-r--r-- | src/backend/optimizer/plan/planner.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c index 819067536f2..de1257d9c22 100644 --- a/src/backend/optimizer/plan/planner.c +++ b/src/backend/optimizer/plan/planner.c @@ -195,12 +195,11 @@ static void add_paths_to_grouping_rel(PlannerInfo *root, RelOptInfo *input_rel, double dNumGroups, List *havingQual); static void add_paths_to_partial_grouping_rel(PlannerInfo *root, RelOptInfo *input_rel, - RelOptInfo *partial_grouped_rel, + RelOptInfo *partially_grouped_rel, AggClauseCosts *agg_partial_costs, grouping_sets_data *gd, bool can_sort, - bool can_hash, - List *havingQual); + bool can_hash); static bool can_parallel_agg(PlannerInfo *root, RelOptInfo *input_rel, RelOptInfo *grouped_rel, const AggClauseCosts *agg_costs); @@ -3838,8 +3837,7 @@ create_grouping_paths(PlannerInfo *root, add_paths_to_partial_grouping_rel(root, input_rel, partially_grouped_rel, &agg_partial_costs, - gd, can_sort, can_hash, - (List *) parse->havingQual); + gd, can_sort, can_hash); } /* Build final grouping paths */ @@ -6224,8 +6222,7 @@ add_paths_to_partial_grouping_rel(PlannerInfo *root, AggClauseCosts *agg_partial_costs, grouping_sets_data *gd, bool can_sort, - bool can_hash, - List *havingQual) + bool can_hash) { Query *parse = root->parse; Path *cheapest_partial_path = linitial(input_rel->partial_pathlist); |