Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/18668 )
Change subject: [tools] Download shell dependencies from PyPI ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/18668/2/shell/packaging/requirements.txt File shell/packaging/requirements.txt: http://gerrit.cloudera.org:8080/#/c/18668/2/shell/packaging/requirements.txt@a2 PS2, Line 2: As noted in https://pypi.org/project/configparser/, importing configparser (lowercase) in Python 2 will use the PyPI library which includes Python 3.2 configparser improvements. However we don't seem to use any of those improvements, as our shell tarball doesn't include the external configparser package. -- To view, visit http://gerrit.cloudera.org:8080/18668 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3b76e112ba9d0db19fae3e8eb15fd54a721f80fd Gerrit-Change-Number: 18668 Gerrit-PatchSet: 2 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Fri, 24 Jun 2022 17:17:15 +0000 Gerrit-HasComments: Yes
