Jason Fehr has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22553
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, 29 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/53/22553/2 -- 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: newchange Gerrit-Change-Id: Idd8121d4fbf7abe12d313f3314377db6f1ec017a Gerrit-Change-Number: 22553 Gerrit-PatchSet: 2 Gerrit-Owner: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
