]> granicus.if.org Git - postgresql/commitdiff
Fix assert with side effects in the new PHJ code.
authorAndres Freund <andres@anarazel.de>
Sun, 24 Dec 2017 10:57:55 +0000 (02:57 -0800)
committerAndres Freund <andres@anarazel.de>
Sun, 24 Dec 2017 10:57:55 +0000 (02:57 -0800)
Instead of asserting the assert just set the value to what it was
supposed to test...

Per coverity.

src/backend/executor/nodeHash.c

index 4284e8682a077d6ddf796d905cf63ffccd15a2be..0a519fae313cc6ec532b522486adaff48a38fbf6 100644 (file)
@@ -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)))
        {