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>
