Qifan Chen has uploaded a new patch set (#2). ( 
http://gerrit.cloudera.org:8080/17999 )

Change subject: IMPALA-11006: Impalad crashes during query cancel tests
......................................................................

IMPALA-11006: Impalad crashes during query cancel tests

This patch addresses impalad crash during query cancel tests. The
root cause is that the wait thread in impalad assumes that the
coordinator or computation results are always available once the
worker thread async_exec_thread_ is joined. In reality, this may
not be true at all once the query is cancelled.

The patch specifically checks the cancel status when it is found
that the coordinator is not available and bails out immediately.
This prevents crash since the subsequent code that assumes the
coordinator or computation results are available is never reached.

Testing:
  1. Ran core tests successfully.

Change-Id: Ia49411f8525734b8d463d9ffbfbce705b90a8d73
---
M be/src/service/client-request-state.cc
1 file changed, 9 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/99/17999/2
--
To view, visit http://gerrit.cloudera.org:8080/17999
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia49411f8525734b8d463d9ffbfbce705b90a8d73
Gerrit-Change-Number: 17999
Gerrit-PatchSet: 2
Gerrit-Owner: Qifan Chen <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Qifan Chen <[email protected]>

Reply via email to