Thomas Marshall has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11465


Change subject: IMPALA-7488: Fix hang in test_cancellation
......................................................................

IMPALA-7488: Fix hang in test_cancellation

test_cancellation runs a impala-shell process with a query specified
then sends a SIGINT and confirms that the shell cancels the query and
exits.

The hang was happening because the shell's signal handler was
incorrectly using the same Thirft connection when calling Close() as
the main shell thread, which is not thread safe.

Testing:
- Ran test_cancellation in a loop 500 times. Previously the hang would
  repro about every 10 runs.

Change-Id: I9c4b570604f7706712eb8e19b1ce69bf35cf15e2
---
M shell/impala_shell.py
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/65/11465/1
--
To view, visit http://gerrit.cloudera.org:8080/11465
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9c4b570604f7706712eb8e19b1ce69bf35cf15e2
Gerrit-Change-Number: 11465
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Marshall <[email protected]>

Reply via email to