Tim Armstrong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/12164


Change subject: IMPALA-8026: Fix #rows accounting for NLJ
......................................................................

IMPALA-8026: Fix #rows accounting for NLJ

Use the same, much simpler, approach used by all other ExecNodes.

Testing:
Manually tested by adding logging to print values of the counter.
Confirmed that the fix led to the counter having the correct
values (instead of values exceeding num_rows_returned_).

Change-Id: I43e2e1d8f85478ff36c8cddc69bac3bdccd2c824
---
M be/src/exec/nested-loop-join-node.cc
1 file changed, 8 insertions(+), 20 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/64/12164/1
--
To view, visit http://gerrit.cloudera.org:8080/12164
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I43e2e1d8f85478ff36c8cddc69bac3bdccd2c824
Gerrit-Change-Number: 12164
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <[email protected]>

Reply via email to