Hello Andrew Sherman, David Knupp, Tim Armstrong, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15219
to look at the new patch set (#2).
Change subject: IMPALA-9384: Improve Impala shell usability by enabling
live_progress in interactive mode
......................................................................
IMPALA-9384: Improve Impala shell usability by enabling live_progress in
interactive mode
In order to improve usability, this patch would like to make Impala shell show
query
processing status by default. The patch enables shell option live_progress when
a user
launches impala shell in the interactive mode. The patch also provides a new
command
line flag “--disable_live_progress”, which allows a user to disable
live_progress at
runtime. In the interactive mode, a user can disable live_progress by either
using
the proposed command line flag or setting the option as False in the config
file. As for
in the non-interactive mode (when the -q or -f options are used), live
reporting is not
supported. Impala-shell will disable live_progress if the mode is detected.
Testing:
- Added and updated tests in test_shell_interactive.py and
test_shell_commandline.py
- Successfully ran all shell related tests
Change-Id: I3765b775f663fa227e59728acffe4d5ea9a5e2d3
---
M shell/impala_shell.py
M shell/impala_shell_config_defaults.py
M shell/option_parser.py
M tests/shell/test_shell_commandline.py
M tests/shell/test_shell_interactive.py
5 files changed, 33 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/19/15219/2
--
To view, visit http://gerrit.cloudera.org:8080/15219
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3765b775f663fa227e59728acffe4d5ea9a5e2d3
Gerrit-Change-Number: 15219
Gerrit-PatchSet: 2
Gerrit-Owner: Alice Fan <[email protected]>
Gerrit-Reviewer: Alice Fan <[email protected]>
Gerrit-Reviewer: Andrew Sherman <[email protected]>
Gerrit-Reviewer: David Knupp <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
