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 (#3).
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 while the query is running. The patch enables shell option
live_progress by default 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 is detected.
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/3
--
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: 3
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]>
