Joe McDonnell has uploaded this change for review. (
http://gerrit.cloudera.org:8080/20087
Change subject: IMPALA-12224: Improve error handling for shell interactive tests
......................................................................
IMPALA-12224: Improve error handling for shell interactive tests
Interactive shell tests can hang waiting for input if the
shell process hits errors or exits. For example, the problems
in the sasl packag seen in IMPALA-12220 cause test_shell_interactive.py
to hang.
This improves the error detection/handling to avoid hangs for
most common shell errors. Specifically, it adds a check for
the impala-shell process exiting, and it adds a check for
a failure to connect to Impala. Both would previous result
in hangs.
Testing:
- Verified test_shell_interactive.py doesn't hang with hand
tests
- Remove a vital import from impala-shell so it exits instantly
- Simulate a connection problem by overwriting the port
with a non-functional port
- Test on Redhat 9 with the IMPALA-12220 issue
Change-Id: I7556fb687e06b41caa538d8c3231ec9f2ad98162
---
M shell/impala_shell.py
M tests/shell/util.py
2 files changed, 57 insertions(+), 9 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/87/20087/1
--
To view, visit http://gerrit.cloudera.org:8080/20087
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7556fb687e06b41caa538d8c3231ec9f2ad98162
Gerrit-Change-Number: 20087
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <[email protected]>
