Henry Robinson has uploaded a new patch set (#2). Change subject: IMPALA-4925: Cancel finstance if query has finished ......................................................................
IMPALA-4925: Cancel finstance if query has finished This patch is a partial fix for the issue where an finst would not detect that it should cancel if the query limit had not been hit. It changes the UpdateExecStatus() RPC to return a cancelled status to an finst if the query has finished because it hit a limit. For certain queries, this allows them to finish much more quickly than they otherwise would. However, there's still a few-second delay for the finst to pick up the cancellation signal, because there UpdateExecStatus() RPC is only called every few seconds. A complete fix would also call CancelInternal() when returned_all_results_ was set to true. That would be a much larger change. The improvement here is to bound the delay between query completion and fragment teardown to a few seconds. Change-Id: I59f45e64978c9ab9914b5c33e86009960b4a88c4 --- M be/src/runtime/coordinator.cc M tests/query_test/test_lifecycle.py 2 files changed, 30 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/87/5987/2 -- To view, visit http://gerrit.cloudera.org:8080/5987 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I59f45e64978c9ab9914b5c33e86009960b4a88c4 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Henry Robinson <he...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
