From: Robert Haas <rhaas@postgresql.org>
Date: Fri, 6 Jan 2017 14:34:26 +0000 (-0500)
Subject: Repair commit b81b5a96f424531b97cdd1dba97d9d1b9c9d372e.
X-Git-Tag: REL_10_BETA1~1146
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0355e6f3107a5be9b5f0873f08527bad361d9d8f;p=postgresql

Repair commit b81b5a96f424531b97cdd1dba97d9d1b9c9d372e.

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
index c16b62467f..dc64b3262a 100644
--- 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));
 }
 
 /*