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 (#5).
Change subject: IMPALA-7031: Cancel action of debug page should not unregister
query
......................................................................
IMPALA-7031: Cancel action of debug page should not unregister query
When a running query is canceled from the Cancel button of
queries debug page (impalad WebUI), 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 the
debug page, and attempts to run an operation on that query handle
but encounters and error instead.
This patch updates the cancel http endpoint to only cancel the query.
Testing:
- Added a test to test_web_pages.py
- Ran all webserver tests
Change-Id: I56983d40e0542bc734ec5a66c339b5131b7b56c8
---
M be/src/service/impala-http-handler.cc
M tests/webserver/test_web_pages.py
2 files changed, 22 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/26/12926/5
--
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: 5
Gerrit-Owner: Alice Fan <fan...@gmail.com>
Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
