Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20095 )
Change subject: IMPALA-12220: pip install ext-py dependencies in the shell tarball ...................................................................... IMPALA-12220: pip install ext-py dependencies in the shell tarball The impala-shell tarball ships its external dependencies by building eggs and including them in the ext-py* directories. On Redhat 9 and Ubuntu 22, the impala-shell tarball encountered a regression where the sasl package could not access its Client class: Error connecting: AttributeError, module 'sasl' has no attribute 'Client' This only occurs when using eggs (which are zip files). The virtualenv installs worked fine. Unpacking the eggs and using the content directly also avoids the problem. This reworks the shell tarball to instead build wheels and install them with 'pip install'. This means that the external dependencies are not packaged in eggs, and this avoids the issue with sasl. This is a minimal change to avoid the issue until the shell tarball build can be reworked more extensively. Testing: - Ran shell tests on Redhat 9 Change-Id: I49403979c559b7f8bbe038865c06db6024468d72 Reviewed-on: http://gerrit.cloudera.org:8080/20095 Reviewed-by: Michael Smith <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M shell/CMakeLists.txt M shell/impala-shell M shell/make_shell_tarball.sh 3 files changed, 81 insertions(+), 54 deletions(-) Approvals: Michael Smith: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/20095 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I49403979c559b7f8bbe038865c06db6024468d72 Gerrit-Change-Number: 20095 Gerrit-PatchSet: 7 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
