Hello Riza Suminto, Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22553
to look at the new patch set (#3).
Change subject: IMPALA-13803: Fix hiveserver2_protocol_version Values in
Workload Management
......................................................................
IMPALA-13803: Fix hiveserver2_protocol_version Values in Workload Management
The workload management tables sys.impala_query_live and
sys.impala_query_log contain different values for the
hiveserver2_protocol_version column. The live table contains a short
string such as "V6" while the log table contains a much longer string
with an unnecessary prefix such as "HIVE_CLI_SERVICE_PROTOCOL_V6".
This patch modifies the value stored in the
hiveserver2_protocol_version column in the sys.impala_query_log table
to match the value stored in the sys.impala_query_live table and the
query profile.
Testing was accomplished by running the following test suites
locally. These tests all call the 'assert_query' function of
workload_management.py which checks the value of the
hiveserver2_protocol_version column. A new test was added to
test_query_live.py which uses HS2 as the client protocol.
* tests/custom_cluster/test_query_live.py
* tests/custom_cluster/test_query_log.py
* tests/custom_cluster/test_workload_mgmt_init.py
Change-Id: Idd8121d4fbf7abe12d313f3314377db6f1ec017a
---
M be/src/exec/system-table-scanner.cc
M be/src/service/query-state-record.cc
M be/src/service/query-state-record.h
M be/src/service/workload-management-worker.cc
M tests/custom_cluster/test_query_live.py
M tests/util/workload_management.py
6 files changed, 30 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/53/22553/3
--
To view, visit http://gerrit.cloudera.org:8080/22553
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idd8121d4fbf7abe12d313f3314377db6f1ec017a
Gerrit-Change-Number: 22553
Gerrit-PatchSet: 3
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
