Hello Quanlong Huang, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20584
to look at the new patch set (#3).
Change subject: IMPALA-12493: Allow retry after failed cancel_query
......................................................................
IMPALA-12493: Allow retry after failed cancel_query
Previously if /cancel_query failed with "Query not yet running",
subsequent attempts to cancel the query would always fail and the query
would never be unregistered (so it would stay in "queries in flight" or
"waiting to be closed" until the coordinator was restarted) because the
QueryDriver thought the query was finalized. Unset finalized_ if
cancelling the query fails to allow retries.
Testing:
- adds a test that canceling a query in CREATED state fails but does not
prevent canceling the query later
- adds a test that canceling a query that later fails results in that
query completing rather than going to "waiting to be closed"
Change-Id: I6e2c8872e91a7ff078cedb13e0771bcfaae6ee24
---
M be/src/runtime/query-driver.cc
M fe/src/main/java/org/apache/impala/service/Frontend.java
M fe/src/main/java/org/apache/impala/util/DebugUtils.java
M tests/webserver/test_web_pages.py
4 files changed, 134 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/84/20584/3
--
To view, visit http://gerrit.cloudera.org:8080/20584
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6e2c8872e91a7ff078cedb13e0771bcfaae6ee24
Gerrit-Change-Number: 20584
Gerrit-PatchSet: 3
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
