Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22700 )
Change subject: IMPALA-13907: Remove reference to create_beeswax_client ...................................................................... IMPALA-13907: Remove reference to create_beeswax_client This patch replace create_beeswax_client() reference to create_hs2_client() or vector-based client creation to prepare towards hs2 test migration. test_session_expiration_with_queued_query is changed to use impala.dbapi directly from Impyla due to limitation in ImpylaHS2Connection. TestAdmissionControllerRawHS2 is migrated to use hs2 as default test protocol. Modify test_query_expiration.py to set query option through client instead of SET query. test_query_expiration is slightly modified due to behavior difference in hs2 ImpylaHS2Connection. Remove remaining reference to BeeswaxConnection.QueryState. Fixed a bug in ImpylaHS2Connection.wait_for_finished_timeout(). Fix some easy flake8 issues caught thorugh this command: git show HEAD --name-only | grep '^tests.*py' \ | xargs -I {} impala-flake8 {} \ | grep -e U100 -e E111 -e E301 -e E302 -e E303 -e F... Testing: - Pass exhaustive tests. Change-Id: I1d84251835d458cc87fb8fedfc20ee15aae18d51 Reviewed-on: http://gerrit.cloudera.org:8080/22700 Reviewed-by: Riza Suminto <riza.sumi...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M tests/common/impala_cluster.py M tests/common/impala_connection.py M tests/common/impala_service.py M tests/common/resource_pool_config.py M tests/custom_cluster/test_admission_controller.py M tests/custom_cluster/test_ai_generate_text.py M tests/custom_cluster/test_blacklist.py M tests/custom_cluster/test_catalog_hms_failures.py M tests/custom_cluster/test_catalog_wait.py M tests/custom_cluster/test_catalogd_ha.py M tests/custom_cluster/test_client_ssl.py M tests/custom_cluster/test_compact_catalog_updates.py M tests/custom_cluster/test_concurrent_ddls.py M tests/custom_cluster/test_coordinators.py M tests/custom_cluster/test_exchange_eos.py M tests/custom_cluster/test_frontend_connection_limit.py M tests/custom_cluster/test_hdfs_fd_caching.py M tests/custom_cluster/test_insert_behaviour.py M tests/custom_cluster/test_local_catalog.py M tests/custom_cluster/test_mem_reservations.py M tests/custom_cluster/test_process_failures.py M tests/custom_cluster/test_query_expiration.py M tests/custom_cluster/test_query_retries.py M tests/custom_cluster/test_refresh_invalid_partition.py M tests/custom_cluster/test_restart_services.py M tests/custom_cluster/test_s3a_access.py M tests/custom_cluster/test_scratch_disk.py M tests/custom_cluster/test_seq_file_filtering.py M tests/custom_cluster/test_session_expiration.py M tests/custom_cluster/test_statestored_ha.py M tests/custom_cluster/test_web_pages.py M tests/metadata/test_compute_stats.py M tests/metadata/test_ddl.py M tests/query_test/test_rows_availability.py M tests/query_test/test_udfs.py M tests/stress/query_retries_stress_runner.py M tests/util/cancel_util.py 37 files changed, 277 insertions(+), 263 deletions(-) Approvals: Riza Suminto: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/22700 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1d84251835d458cc87fb8fedfc20ee15aae18d51 Gerrit-Change-Number: 22700 Gerrit-PatchSet: 12 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>