From c5dcbc32646e8e4865307bb0525a143da573e240 Mon Sep 17 00:00:00 2001 From: Kyotaro Horiguchi Date: Wed, 8 May 2019 13:01:01 +0900 Subject: [PATCH] Give correct number of arguments to combine function. Combine function's first parameter is used as transition value. The correct number of transition input for the function is not the number of arguments but of transition input. --- src/backend/executor/nodeAgg.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/backend/executor/nodeAgg.c b/src/backend/executor/nodeAgg.c index d01fc4f52e..e13a8cb304 100644 --- a/src/backend/executor/nodeAgg.c +++ b/src/backend/executor/nodeAgg.c @@ -2912,7 +2912,8 @@ build_pertrans_for_aggref(AggStatePerTrans pertrans, pertrans->aggtranstype = aggtranstype; /* Detect how many arguments to pass to the transfn */ - if (AGGKIND_IS_ORDERED_SET(aggref->aggkind)) + if (AGGKIND_IS_ORDERED_SET(aggref->aggkind) || + DO_AGGSPLIT_COMBINE(aggstate->aggsplit)) pertrans->numTransInputs = numInputs; else pertrans->numTransInputs = numArguments; -- 2.16.3