Michael Smith has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/19797


Change subject: IMPALA-12094: Fix impala shell summary command
......................................................................

IMPALA-12094: Fix impala shell summary command

Fix various quality-of-life issues with the 'summary' command:
- update regex to correctly match query ID for handling "Query id ...
  not found" errors
- fail the command rather than exiting the shell when 'summary' is
  called with an incorrect argument (such as 'summary 1')
- provide a useful message rather than print an exception when 'summary
  original' is invoked with no failed queries

Testing:
- added new tests for the 'summary' command

Change-Id: I7523d45b27e5e63e1f962fb1f6ebb4f0adc85213
---
M shell/impala_shell.py
M tests/shell/test_shell_commandline.py
2 files changed, 24 insertions(+), 2 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7523d45b27e5e63e1f962fb1f6ebb4f0adc85213
Gerrit-Change-Number: 19797
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Smith <[email protected]>

Reply via email to