Repair commit b81b5a9.

robertmhaas · robertmhaas · commit 0355e6f3107a · 2017-01-06T09:34:26.000-05:00
This commit purported to use a variable hash seed for Partial
HashAggregate, but actually did the opposite - it made us use a
variable seed for any HashAggregate that is NOT partial.  Woops.
diff --git a/src/backend/executor/nodeAgg.c b/src/backend/executor/nodeAgg.c
@@ -1738,7 +1738,7 @@ build_hash_table(AggState *aggstate)
 											  additionalsize,
 							 aggstate->aggcontexts[0]->ecxt_per_tuple_memory,
 											  tmpmem,
-								  !DO_AGGSPLIT_SKIPFINAL(aggstate->aggsplit));
+								  DO_AGGSPLIT_SKIPFINAL(aggstate->aggsplit));
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -1738,7 +1738,7 @@ build_hash_table(AggState *aggstate)`
`1738`	`1738`	`additionalsize,`
`1739`	`1739`	`aggstate->aggcontexts[0]->ecxt_per_tuple_memory,`
`1740`	`1740`	`tmpmem,`
`1741`		`- !DO_AGGSPLIT_SKIPFINAL(aggstate->aggsplit));`
	`1741`	`+ DO_AGGSPLIT_SKIPFINAL(aggstate->aggsplit));`
`1742`	`1742`	`}`
`1743`	`1743`
`1744`	`1744`	`/*`