impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thomas Tauber-Marshall (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-1473: Incorrect cardinality in exec summary for exchange
Date Tue, 11 Oct 2016 00:08:43 GMT
Thomas Tauber-Marshall has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/4679

Change subject: IMPALA-1473: Incorrect cardinality in exec summary for exchange
......................................................................

IMPALA-1473: Incorrect cardinality in exec summary for exchange

When there is a merging exchange with a limit, we may copy rows into
the output batch beyond the limit. In this case, we currently update
the output batch's size to reflect the limit, but we also need to
update ExecNode::num_rows_returned_ or the exec summary may show
that the exchange node returned more rows than it really did.

Change-Id: I386719370386c9cff09b8b35d15dc712dc6480aa
---
M be/src/exec/exchange-node.cc
A tests/query_test/test_exec_summary.py
2 files changed, 73 insertions(+), 0 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I386719370386c9cff09b8b35d15dc712dc6480aa
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <tmarshall@cloudera.com>

Mime
View raw message