Hello Bikramjeet Vig, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12926

to look at the new patch set (#9).

Change subject: IMPALA-7031: Add additional info to query canceled from http 
endpoint
......................................................................

IMPALA-7031: Add additional info to query canceled from http endpoint

When a running query is canceled from the http endpoint, the query will
be unregistered. This can result in a scenario where a client that has
a reference to the query handle is unaware that the query has been
unregistered via http endpoint, and attempts to run an operation on
that query handle but encounters "Invalid query handle" error.
From the client/user prospective the error is not indicative, so the
patch added additional information to error to notify that the query is
already unregistered.

Testing:
- Added a test to test_web_pages.py
- Ran all webserver tests and all tests have been changed

Change-Id: I56983d40e0542bc734ec5a66c339b5131b7b56c8
---
M be/src/service/impala-beeswax-server.cc
M be/src/service/impala-hs2-server.cc
M be/src/service/impala-server.cc
M be/src/service/impala-server.h
M tests/hs2/test_hs2.py
M tests/query_test/test_cancellation.py
M tests/shell/test_shell_commandline.py
M tests/stress/concurrent_select.py
M tests/webserver/test_web_pages.py
9 files changed, 75 insertions(+), 26 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/26/12926/9
--
To view, visit http://gerrit.cloudera.org:8080/12926
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I56983d40e0542bc734ec5a66c339b5131b7b56c8
Gerrit-Change-Number: 12926
Gerrit-PatchSet: 9
Gerrit-Owner: Alice Fan <fan...@gmail.com>
Gerrit-Reviewer: Alice Fan <fan...@gmail.com>
Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>

Reply via email to