impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Internal Jenkins (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-3002/IMPALA-1473: Cardinality observability cleanup
Date Sat, 15 Oct 2016 01:25:52 GMT
Internal Jenkins has submitted this change and it was merged.

Change subject: IMPALA-3002/IMPALA-1473: Cardinality observability cleanup

IMPALA-3002/IMPALA-1473: Cardinality observability cleanup

The shell prints an incorrect value for '#Rows' in the exec
summary for broadcast nodes due to incorrect logic around
whether to use max or agg stats. This patch makes the behavior
consistent with the way the be treats exec summaries in This incorrect logic was also duplicated in
the impala_beeswax test framework.

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

Additionally, PlanFragmentExecutor::GetNext does not update
rows_produced_counter_ in some cases, leading the runtime profile
to display an incorrect value for 'RowsProduced'.

Change-Id: I386719370386c9cff09b8b35d15dc712dc6480aa
Reviewed-by: Matthew Jacobs <>
Tested-by: Internal Jenkins
M be/src/exec/
M be/src/runtime/
M shell/
M tests/beeswax/
A tests/query_test/
5 files changed, 63 insertions(+), 7 deletions(-)

  Matthew Jacobs: Looks good to me, approved
  Internal Jenkins: Verified

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: I386719370386c9cff09b8b35d15dc712dc6480aa
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-Reviewer: Thomas Tauber-Marshall <>

View raw message