Hello Tim Armstrong, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12530
to look at the new patch set (#8).
Change subject: IMPALA-8158: Retrieve thrift profiles through Impyla 0.15.0
......................................................................
IMPALA-8158: Retrieve thrift profiles through Impyla 0.15.0
This change updates Impyla to 0.15.0 and then uses Impyla to retrieve
thrift profiles through the HS2 api.
Unfortunately, some of the current usages of get_thrift_profile rely on
the Beeswax query states and the ImpylaHS2Connection does not have the
required functionality yet. We will have to update these in a future
change, once we unified the query states.
This change also adds a self-contained test for IMPALA-2063
This change also makes a change to update the query operation state to
ERROR_STATE upon the client cancellation. Previously we would wait to
update the state until the client disconnected, which led to a race
where the client could disconnect, then ask for a state and still see
STATE_RUNNING while backend was handling the disconnect event.
Change-Id: I769a99f0843297dd2b20f2f5b1a9046c97bb131e
---
M be/src/service/impala-hs2-server.cc
M infra/python/deps/compiled-requirements.txt
M tests/common/impala_connection.py
M tests/common/impala_service.py
M tests/common/impala_test_suite.py
M tests/query_test/test_cancellation.py
M tests/query_test/test_observability.py
7 files changed, 92 insertions(+), 86 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/30/12530/8
--
To view, visit http://gerrit.cloudera.org:8080/12530
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I769a99f0843297dd2b20f2f5b1a9046c97bb131e
Gerrit-Change-Number: 12530
Gerrit-PatchSet: 8
Gerrit-Owner: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
