Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/20617 )
Change subject: IMPALA-12515: Build modules for extra pythons ...................................................................... Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/20617/1/bin/impala-config.sh File bin/impala-config.sh: http://gerrit.cloudera.org:8080/#/c/20617/1/bin/impala-config.sh@228 PS1, Line 228: # Additional Python versions to package for > Can you expand this comment a bit? Talk about how this is about the shell t Done http://gerrit.cloudera.org:8080/#/c/20617/1/shell/CMakeLists.txt File shell/CMakeLists.txt: http://gerrit.cloudera.org:8080/#/c/20617/1/shell/CMakeLists.txt@47 PS1, Line 47: if (${PYTHON_EXE} EQUAL $ENV{IMPALA_SYSTEM_PYTHON2}) : set(PYTHON2_VENV ${VENV}) : set(PYTHON2_PIP_CACHE ${PIP_CACHE}) : elseif (${PYTHON_EXE} EQUAL $ENV{IMPALA_SYSTEM_PYTHON3}) : set(PYTHON3_VENV ${VENV}) : set(PYTHON3_PIP_CACHE ${PIP_CACHE}) : endif() > We also set PYTHON2_VENV etc down below next to the shell_pythonX_install t Done http://gerrit.cloudera.org:8080/#/c/20617/1/shell/impala-shell File shell/impala-shell: http://gerrit.cloudera.org:8080/#/c/20617/1/shell/impala-shell@51 PS1, Line 51: echo "Python ${PYTHON_VERSION} is not supported, try 'pip install impala-shell'." > I'm thinking about what exactly we want in the error message. We want to av Done http://gerrit.cloudera.org:8080/#/c/20617/1/shell/make_shell_tarball.sh File shell/make_shell_tarball.sh: http://gerrit.cloudera.org:8080/#/c/20617/1/shell/make_shell_tarball.sh@150 PS1, Line 150: --upgrade > What does the "--upgrade" do? Do we need it? It essentially suppresses a WARNING WARNING: Target directory /home/michael/Impala/shell/build/impala-shell-4.4.0-SNAPSHOT/ext-py2.7/thrift_sasl already exists. Specify --upgrade to force replacement. when the list of python interpreters includes duplicates for the same minor version (such as python3 and python3.8). I've added a comment. -- To view, visit http://gerrit.cloudera.org:8080/20617 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I13720a9e3c50f348bef41f5e91f810204e416f13 Gerrit-Change-Number: 20617 Gerrit-PatchSet: 1 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Comment-Date: Wed, 25 Oct 2023 20:37:48 +0000 Gerrit-HasComments: Yes
