impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "anujphadke (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4866: Hash join node does not apply limits correctly
Date Tue, 09 May 2017 01:15:15 GMT
anujphadke has posted comments on this change.

Change subject: IMPALA-4866: Hash join node does not apply limits correctly

Patch Set 2:

Commit Message:

PS1, Line 11: exchange
> typo
File be/src/exec/

Line 506:     DCHECK(status.ok());
> there are a lot of other places that this loop can exit. Can't the out_batc
I have moved the check outside the while loop, so that this gets invoked everytime we break
out of it. 
Ran a bunch of queries and tried to verify that the limit gets applied correctly Found a few
cases where num_rows_returned was not incremented and the ReachedLimit check was not correctly

PS1, Line 580: 
> >= ?
Removed this check now. Please  ignore.

Line 581:       DCHECK(current_probe_row_ == NULL);
> I think you need to update num_rows_returned_ as well
Removed this check. Please  ignore.
File tests/common/

PS1, Line 114: # Don't run with NUM_NODES=1 due to IMPALA-561
             : # ALL_CLUSTER_SIZES = [0, 1]
> update comment
Ran the private tests by changing this value ( but without the fix to correctly apply the
limits). Current tests did not catch this. 
I am reverting it back and adding a few tests which can catch this issue.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I414124f8bb6f8b2af2df468e1c23418d05a0e29f
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: anujphadke <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-Reviewer: anujphadke <>
Gerrit-HasComments: Yes

View raw message