Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/15624 )
Change subject: Use Python from the toolchain for Impala ...................................................................... Patch Set 8: (8 comments) http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py File infra/python/tmp.py: http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py@29 PS8, Line 29: except: flake8: E722 do not use bare except' http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py@33 PS8, Line 33: import ssl flake8: E402 module level import not at top of file http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py@33 PS8, Line 33: import ssl flake8: F401 'ssl' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py@34 PS8, Line 34: from httplib import HTTPConnection flake8: E402 module level import not at top of file http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py@34 PS8, Line 34: from httplib import HTTPConnection flake8: F401 'httplib.HTTPConnection' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py@35 PS8, Line 35: from urllib2 import HTTPSHandler flake8: E402 module level import not at top of file http://gerrit.cloudera.org:8080/#/c/15624/8/infra/python/tmp.py@35 PS8, Line 35: from urllib2 import HTTPSHandler flake8: F401 'urllib2.HTTPSHandler' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/8/tests/common/impala_test_suite.py File tests/common/impala_test_suite.py: http://gerrit.cloudera.org:8080/#/c/15624/8/tests/common/impala_test_suite.py@33 PS8, Line 33: import sys flake8: F401 'sys' imported but unused -- To view, visit http://gerrit.cloudera.org:8080/15624 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic7b40cef89cfb3b467b61b2d54a94e708642882b Gerrit-Change-Number: 15624 Gerrit-PatchSet: 8 Gerrit-Owner: Laszlo Gaal <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 08 Apr 2020 17:09:24 +0000 Gerrit-HasComments: Yes
