Hello Fang-Yu Rao, Yida Wu, Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22731
to look at the new patch set (#4).
Change subject: IMPALA-13930: ImpylaHS2Connection should only open cursor as
needed
......................................................................
IMPALA-13930: ImpylaHS2Connection should only open cursor as needed
ImpylaHS2Connection open a cursor (and HS2 session) as it connect,
followed by running a "SET ALL" query to populate the default query
options.
This patch changes the behavior of ImpylaHS2Connection to open the
default cursor only when querying is needed for the first time. This
helps preserve assertions for a test that is sensitive about client
connection, like IMPALA-13925. Changed the default query option parsing
technique from running a "SET ALL" query to parsing newly instantiated
TQueryOptions object.
Fix test_query_profile_contains_query_compilation_metadata_cached_event
slightly by setting the 'sync_ddl' option because the test is flaky
without it.
Tweak test_max_hs2_sessions_per_user to run queries so that sessions
will open.
Testing:
Run and pass a few pytests such as test_runtime_filters.py,
test_observability.py, test_admission_controller.py, and
test_session_expiration.py.
Change-Id: I9d5e3e5c11ad386b7202431201d1a4cff46cbff5
---
M tests/common/impala_connection.py
M tests/custom_cluster/test_session_expiration.py
M tests/query_test/test_observability.py
3 files changed, 58 insertions(+), 35 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/31/22731/4
--
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: newpatchset
Gerrit-Change-Id: I9d5e3e5c11ad386b7202431201d1a4cff46cbff5
Gerrit-Change-Number: 22731
Gerrit-PatchSet: 4
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Fang-Yu Rao <fangyu....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Yida Wu <wydbaggio...@gmail.com>
