From 4e2970f8807f1ccfc8029979a70dc80ee102ce48 Mon Sep 17 00:00:00 2001 From: Andres Freund Date: Sun, 24 Dec 2017 02:57:55 -0800 Subject: [PATCH] Fix assert with side effects in the new PHJ code. Instead of asserting the assert just set the value to what it was supposed to test... Per coverity. --- src/backend/executor/nodeHash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/executor/nodeHash.c b/src/backend/executor/nodeHash.c index 4284e8682a..0a519fae31 100644 --- a/src/backend/executor/nodeHash.c +++ b/src/backend/executor/nodeHash.c @@ -1275,7 +1275,7 @@ ExecParallelHashRepartitionFirst(HashJoinTable hashtable) dsa_pointer chunk_shared; HashMemoryChunk chunk; - Assert(hashtable->nbatch = hashtable->parallel_state->nbatch); + Assert(hashtable->nbatch == hashtable->parallel_state->nbatch); while ((chunk = ExecParallelHashPopChunkQueue(hashtable, &chunk_shared))) { -- 2.40.0