Hello Jason Fehr, Csaba Ringhofer, Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22586
to look at the new patch set (#3).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/86/22586/3
--
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: newpatchset
Gerrit-Change-Id: Id6b56024fbfcea1ff005c34cd146d16e67cb6fa1
Gerrit-Change-Number: 22586
Gerrit-PatchSet: 3
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]>
