Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14998 )

Change subject: IMPALA-6393 live_summary and live_progress are not supported in 
impalarc
......................................................................


Patch Set 1:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/14998/1/shell/impala_shell.py
File shell/impala_shell.py:

http://gerrit.cloudera.org:8080/#/c/14998/1/shell/impala_shell.py@154
PS1, Line 154:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/14998/1/shell/impala_shell.py@155
PS1, Line 155:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/14998/1/shell/impala_shell.py@1015
PS1, Line 1015: ;
flake8: E703 statement ends with a semicolon


http://gerrit.cloudera.org:8080/#/c/14998/1/shell/impala_shell_config_defaults.py
File shell/impala_shell_config_defaults.py:

http://gerrit.cloudera.org:8080/#/c/14998/1/shell/impala_shell_config_defaults.py@42
PS1, Line 42:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/14998/1/shell/impala_shell_config_defaults.py@43
PS1, Line 43:
flake8: E203 whitespace before ':'


http://gerrit.cloudera.org:8080/#/c/14998/1/tests/shell/test_shell_interactive.py
File tests/shell/test_shell_interactive.py:

http://gerrit.cloudera.org:8080/#/c/14998/1/tests/shell/test_shell_interactive.py@508
PS1, Line 508: #
flake8: E265 block comment should start with '# '



--
To view, visit http://gerrit.cloudera.org:8080/14998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I68e7ac008729fc30fd5d67ec973231334c68be9d
Gerrit-Change-Number: 14998
Gerrit-PatchSet: 1
Gerrit-Owner: Wenzhe Zhou <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Fri, 10 Jan 2020 03:37:46 +0000
Gerrit-HasComments: Yes

Reply via email to