database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-04-22 23:02:54 +03:00
Code

Remove 'target' from GroupPathExtraData.

Browse Source 
It's not needed.

Jeevan Chalke

Discussion: http://postgr.es/m/CAM2+6=XPWujjmj5zUaBTGDoB38CemwcPmjkRy0qOcsQj_V+2sQ@mail.gmail.com
This commit is contained in:
Robert Haas 2018-03-29 16:17:18 -04:00
parent 11cf92f6e2
commit c1de1a3a8b
2 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/backend/optimizer/plan/planner.c
View File

@ -3744,7 +3744,6 @@ create_grouping_paths(PlannerInfo *root,
flags |= GROUPING_CAN_PARTIAL_AGG;
extra.flags = flags;
extra.target = target;
extra.target_parallel_safe = target_parallel_safe;
extra.havingQual = parse->havingQual;
extra.targetList = parse->targetList;
@ -7029,7 +7028,7 @@ create_partitionwise_grouping_paths(PlannerInfo *root,
int cnt_parts;
List *grouped_live_children = NIL;
List *partially_grouped_live_children = NIL;
PathTarget *target = extra->target;
PathTarget *target = grouped_rel->reltarget;
Assert(patype != PARTITIONWISE_AGGREGATE_NONE);
Assert(patype != PARTITIONWISE_AGGREGATE_PARTIAL ||
@ -7062,7 +7061,6 @@ create_partitionwise_grouping_paths(PlannerInfo *root,
adjust_appendrel_attrs(root,
(Node *) target->exprs,
nappinfos, appinfos);
child_extra.target = child_target;
/* Translate havingQual and targetList. */
child_extra.havingQual = (Node *)

2
src/include/nodes/relation.h
View File

@ -2340,7 +2340,6 @@ typedef enum
* have been initialized.
* agg_partial_costs gives partial aggregation costs.
* agg_final_costs gives finalization costs.
* target is the PathTarget to be used while creating paths.
* target_parallel_safe is true if target is parallel safe.
* havingQual gives list of quals to be applied after aggregation.
* targetList gives list of columns to be projected.
@ -2355,7 +2354,6 @@ typedef struct
AggClauseCosts agg_final_costs;
/* Data which may differ across partitions. */
PathTarget *target;
bool target_parallel_safe;
Node *havingQual;
List *targetList;