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 4: (12 comments) http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/bootstrap_virtualenv.py File infra/python/bootstrap_virtualenv.py: http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/bootstrap_virtualenv.py@51 PS4, Line 51: import ssl flake8: F401 'ssl' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/bootstrap_virtualenv.py@52 PS4, Line 52: from httplib import HTTPConnection flake8: F401 'httplib.HTTPConnection' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/bootstrap_virtualenv.py@53 PS4, Line 53: from urllib2 import HTTPSHandler flake8: F401 'urllib2.HTTPSHandler' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/bootstrap_virtualenv.py@92 PS4, Line 92: o flake8: F841 local variable 'output' is assigned to but never used http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/bootstrap_virtualenv.py@211 PS4, Line 211: + flake8: E226 missing whitespace around arithmetic operator http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py File infra/python/tmp.py: http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py@29 PS4, Line 29: except: flake8: E722 do not use bare except' http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py@33 PS4, Line 33: import ssl flake8: E402 module level import not at top of file http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py@33 PS4, Line 33: import ssl flake8: F401 'ssl' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py@34 PS4, Line 34: from httplib import HTTPConnection flake8: E402 module level import not at top of file http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py@34 PS4, Line 34: from httplib import HTTPConnection flake8: F401 'httplib.HTTPConnection' imported but unused http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py@35 PS4, Line 35: from urllib2 import HTTPSHandler flake8: E402 module level import not at top of file http://gerrit.cloudera.org:8080/#/c/15624/4/infra/python/tmp.py@35 PS4, Line 35: from urllib2 import HTTPSHandler flake8: F401 'urllib2.HTTPSHandler' 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: 4 Gerrit-Owner: Laszlo Gaal <laszlo.g...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Comment-Date: Thu, 02 Apr 2020 14:17:38 +0000 Gerrit-HasComments: Yes