From 6d050bc9f872983708efe15ba88034d1c6c85ef4 Mon Sep 17 00:00:00 2001 From: Andres Freund Date: Tue, 27 Sep 2016 17:20:37 -0700 Subject: [PATCH] Initialize count_agg_clauses argument to 0. count_agg_clause *adds* the cost of the aggregates to the state variable, it doesn't reinitialize it. That is intentional, as it is used to incrementally add costs in some places. --- src/backend/distributed/planner/multi_master_planner.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/backend/distributed/planner/multi_master_planner.c b/src/backend/distributed/planner/multi_master_planner.c index 05a522e6e..5efd68fc5 100644 --- a/src/backend/distributed/planner/multi_master_planner.c +++ b/src/backend/distributed/planner/multi_master_planner.c @@ -134,6 +134,7 @@ BuildAggregatePlan(Query *masterQuery, Plan *subPlan) Assert(masterQuery->hasAggs || masterQuery->groupClause); aggregateTargetList = masterQuery->targetList; + memset(&aggregateCosts, 0, sizeof(AggClauseCosts)); count_agg_clauses(NULL, (Node *) aggregateTargetList, &aggregateCosts); /*