Tim Armstrong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/9441


Change subject: IMPALA-6582: fix test_multiline_queries_in_history
......................................................................

IMPALA-6582: fix test_multiline_queries_in_history

The semicolon was in the wrong place in one of the test queries and
the failure was swallowed silently.

This meant that one fewer prompt was displayed than expected. This
didn't cause a test failure because the prompt regex also matched the
"Connected to host:port" message printed in the shell preamble. I'm
unsure why this would cause the test failure but my best theory is that
in the failure case, the "Connected" and prompt messages are both
buffered when we evaluate the first prompt regex, and the regex swallows
up the whole input, rather than just the first instance.

Testing:
Tightened up the prompt regex and checked that the query actually
executed successfully. With these improvements, the broken query
text caused a test failure.

I looped the test for a while to make sure it was robust.

Change-Id: If917bbc8e87b83c188b6d5e1acad912892b8c6fe
---
M tests/shell/test_shell_interactive.py
1 file changed, 3 insertions(+), 2 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If917bbc8e87b83c188b6d5e1acad912892b8c6fe
Gerrit-Change-Number: 9441
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>

Reply via email to