Wenzhe Zhou has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/18439 )
Change subject: IMPALA-11263: Coordinator hang when cancelling a query ...................................................................... IMPALA-11263: Coordinator hang when cancelling a query In a rare case, callback Coordinator::BackendState::ExecCompleteCb() is not called for the corresponding ExecQueryFInstances RPC when the RPC is cancelled. This causes coordinator to wait indefinitely when calling Coordinator::BackendState::Cancel() to cancel a fragment instance. This patch adds timeout for BackendState::WaitOnExecLocked() so that coordinator will not be blocked indefinitely when cancelling a query. Testing: - Added a test case to simulate the callback missing when a query is failed. Verified that the coordinator would hang without the fixing, and would not hang with the fixing. - Passed exhaustive-debug tests. Change-Id: I915511afe2df3017cbbf37f6aff3c5ff7f5473be --- M be/src/runtime/coordinator-backend-state.cc M be/src/runtime/coordinator-backend-state.h M tests/custom_cluster/test_rpc_timeout.py 3 files changed, 153 insertions(+), 96 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/39/18439/3 -- To view, visit http://gerrit.cloudera.org:8080/18439 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I915511afe2df3017cbbf37f6aff3c5ff7f5473be Gerrit-Change-Number: 18439 Gerrit-PatchSet: 3 Gerrit-Owner: Wenzhe Zhou <wz...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>