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