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

Change subject: IMPALA-8595: Support TLSv1.2 with Python < 2.7.9 in shell
......................................................................


Patch Set 6:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/13457/6/tests/custom_cluster/test_client_ssl.py
File tests/custom_cluster/test_client_ssl.py:

http://gerrit.cloudera.org:8080/#/c/13457/6/tests/custom_cluster/test_client_ssl.py@28
PS6, Line 28: import platform
flake8: F401 'platform' imported but unused


http://gerrit.cloudera.org:8080/#/c/13457/6/tests/custom_cluster/test_client_ssl.py@41
PS6, Line 41: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/13457/6/tests/custom_cluster/test_client_ssl.py@41
PS6, Line 41: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/13457/6/tests/custom_cluster/test_client_ssl.py@43
PS6, Line 43: ,
flake8: E231 missing whitespace after ','


http://gerrit.cloudera.org:8080/#/c/13457/6/tests/custom_cluster/test_client_ssl.py@43
PS6, Line 43: ,
flake8: E231 missing whitespace after ','



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3fb6510f4b556bd8c6b1e86380379aba8be4b805
Gerrit-Change-Number: 13457
Gerrit-PatchSet: 6
Gerrit-Owner: Robbie Zhang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Fri, 31 May 2019 03:40:28 +0000
Gerrit-HasComments: Yes

Reply via email to