postgres.git / summary / log / commit / refs

commit    4e2970f8807f1ccfc8029979a70dc80ee102ce48
Author:   Andres Freund <andres@anarazel.de>
Date:     Sun Dec 24 10:57:55 2017 +0000
First released in: 11.0

    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 4284e8682a0..0a519fae313 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)))
 	{


[parent: c4c2885cbb18]