Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/23597


Change subject: IMPALA-14491: Fix run-workload.py's handling of HS2's exec 
summary
......................................................................

IMPALA-14491: Fix run-workload.py's handling of HS2's exec summary

Recently, we switched bin/run-workload.py to use HS2. It turns
out that the HS2 client code is not producing the same data
structure for the exec summary. report_benchmark_results.py
relies on that data structure and fails for HS2.

This changes the HS2 client code to use the same representation
as the beeswax. There is already a function that does this
conversion (build_summary_table_from_thrift) for our regular
tests, so this reuses that function.

Testing:
 - Ran bin/run-workload.py twice to produce json files and
   processed them with report_benchmark_results.py. This
   failed before the change and passed afterward.

Change-Id: I0a041bdebe748b6b3a05b552584e0ca2327cff67
---
M tests/performance/query_exec_functions.py
1 file changed, 2 insertions(+), 1 deletion(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a041bdebe748b6b3a05b552584e0ca2327cff67
Gerrit-Change-Number: 23597
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <[email protected]>

Reply via email to