Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14715 )
Change subject: IMPALA-4618: Fixing #Hosts and adding #Instances in exec summary ...................................................................... IMPALA-4618: Fixing #Hosts and adding #Instances in exec summary When mt_dop > 0, the summary is reporting the number of fragment instances, instead of the number of hosts as the header would imply. This commit fixes the issue so the number of hosts will be shown under the #Hosts column. The commit also adds an #Inst column where the number of instances are shown (current behaviour). Tests: * Changed profile tests with mt_dop > 0. * Updated benchmark tests and shell tests accordingly. Change-Id: I3bdf9a06d9bd842b2397cd16c28294b6bec7af69 Reviewed-on: http://gerrit.cloudera.org:8080/14715 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/runtime/coordinator.cc M be/src/util/summary-util.cc M common/thrift/ExecStats.thrift M shell/impala_client.py M shell/impala_shell.py M testdata/workloads/functional-query/queries/QueryTest/mt-dop-parquet-scheduling.test M tests/benchmark/report_benchmark_results.py M tests/shell/test_shell_commandline.py 8 files changed, 65 insertions(+), 41 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/14715 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3bdf9a06d9bd842b2397cd16c28294b6bec7af69 Gerrit-Change-Number: 14715 Gerrit-PatchSet: 6 Gerrit-Owner: Norbert Luksa <norbert.lu...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Norbert Luksa <norbert.lu...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>