Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22586 )
Change subject: IMPALA-13835: Remove reference to protocol-specific states ...................................................................... IMPALA-13835: Remove reference to protocol-specific states With IMPALA-13682 merged, checking for query state can be done via wait_for_impala_state(), wait_for_any_impala_state() and other helper methods of ImpalaConnection. This patch remove all reference to protocol-specific states such as BeeswaxService.QueryState. Also fix flake8 errors and unused variable in modified test files. Testing: - Run and pass all affected tests. Change-Id: Id6b56024fbfcea1ff005c34cd146d16e67cb6fa1 Reviewed-on: http://gerrit.cloudera.org:8080/22586 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M tests/custom_cluster/test_blacklist.py M tests/custom_cluster/test_catalogd_ha.py M tests/custom_cluster/test_events_custom_configs.py M tests/custom_cluster/test_executor_groups.py M tests/custom_cluster/test_kudu.py M tests/custom_cluster/test_process_failures.py M tests/custom_cluster/test_query_live.py M tests/custom_cluster/test_query_log.py M tests/custom_cluster/test_query_retries.py M tests/custom_cluster/test_restart_services.py M tests/custom_cluster/test_result_spooling.py M tests/custom_cluster/test_statestored_ha.py M tests/metadata/test_ddl.py M tests/metadata/test_load.py M tests/query_test/test_cancellation.py M tests/query_test/test_fetch.py M tests/query_test/test_result_spooling.py M tests/webserver/test_web_pages.py 18 files changed, 136 insertions(+), 148 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22586 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id6b56024fbfcea1ff005c34cd146d16e67cb6fa1 Gerrit-Change-Number: 22586 Gerrit-PatchSet: 5 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[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]>
