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

Change subject: IMPALA-10309: Use sleep time from a Retry-After header in 
Impala Shell
......................................................................


Patch Set 2:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16702/2/shell/ImpalaHttpClient.py
File shell/ImpalaHttpClient.py:

http://gerrit.cloudera.org:8080/#/c/16702/2/shell/ImpalaHttpClient.py@31
PS2, Line 31: from shell_exceptions import RPCException, HttpError
flake8: F401 'shell_exceptions.RPCException' imported but unused


http://gerrit.cloudera.org:8080/#/c/16702/2/shell/shell_exceptions.py
File shell/shell_exceptions.py:

http://gerrit.cloudera.org:8080/#/c/16702/2/shell/shell_exceptions.py@55
PS2, Line 55: class HttpError(Exception):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/16702/2/tests/custom_cluster/test_hs2_fault_injection.py
File tests/custom_cluster/test_hs2_fault_injection.py:

http://gerrit.cloudera.org:8080/#/c/16702/2/tests/custom_cluster/test_hs2_fault_injection.py@194
PS2, Line 194: }
flake8: E501 line too long (92 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2b4226e7723d585d61deb4d1d6777aac901bfd93
Gerrit-Change-Number: 16702
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Sherman <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Tue, 10 Nov 2020 17:21:44 +0000
Gerrit-HasComments: Yes

Reply via email to