From c1de1a3a8b93a61e8264484a10a482156026d12c Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Thu, 29 Mar 2018 16:17:18 -0400 Subject: [PATCH] Remove 'target' from GroupPathExtraData. It's not needed. Jeevan Chalke Discussion: http://postgr.es/m/CAM2+6=XPWujjmj5zUaBTGDoB38CemwcPmjkRy0qOcsQj_V+2sQ@mail.gmail.com --- src/backend/optimizer/plan/planner.c | 4 +--- src/include/nodes/relation.h | 2 -- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c index 359f3fc974a..60e2f21d987 100644 --- a/src/backend/optimizer/plan/planner.c +++ b/src/backend/optimizer/plan/planner.c @@ -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 *) diff --git a/src/include/nodes/relation.h b/src/include/nodes/relation.h index abbbda9e91f..2b4f773c70a 100644 --- a/src/include/nodes/relation.h +++ b/src/include/nodes/relation.h @@ -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;