Impala Public Jenkins has submitted this change and it was merged. ( 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 Reviewed-on: http://gerrit.cloudera.org:8080/19797 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M shell/impala_shell.py M tests/shell/test_shell_commandline.py 2 files changed, 24 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I7523d45b27e5e63e1f962fb1f6ebb4f0adc85213 Gerrit-Change-Number: 19797 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
