Riza Suminto has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22731
Change subject: IMPALA-13930: ImpylaHS2Connection should only open cursor as needed ...................................................................... IMPALA-13930: ImpylaHS2Connection should only open cursor as needed ImpylaHS2Connection used to open a cursor (and HS2 session) as it connect, followed by running "SET ALL" query to populate default query options. This patch change the behavior of ImpylaHS2Connection to open cursor only when querying is needed for the very first time. This help preserve assertion for test that sensitive about client connection, like IMPALA-13925. Changed the default query option parsing technique from running "SET ALL" query to parsing newly instantiated TQueryOptions object. Fix test_query_profile_contains_query_compilation_metadata_cached_event slightly with 'sync_ddl' options because the test is flaky without it. Testing: Run and pass few pytest such as test_runtime_filters.py, test_observability.py, and test_admission_controller.py. Change-Id: I9d5e3e5c11ad386b7202431201d1a4cff46cbff5 --- M tests/common/impala_connection.py M tests/query_test/test_observability.py 2 files changed, 52 insertions(+), 33 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/22731/1 -- To view, visit http://gerrit.cloudera.org:8080/22731 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I9d5e3e5c11ad386b7202431201d1a4cff46cbff5 Gerrit-Change-Number: 22731 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
