[ 
https://issues.apache.org/jira/browse/IMPALA-11399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17906556#comment-17906556
 ] 

ASF subversion and git services commented on IMPALA-11399:
----------------------------------------------------------

Commit aefd1b0920150feff31922a6979affc005a6a7d4 in impala's branch 
refs/heads/master from Joe McDonnell
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=aefd1b092 ]

IMPALA-13551: Produce the shell tarball by pip installing impala-shell

Currently, the shell tarball maintains its own packaging code
and directory layout. This is very complicated and currently has
several Python packages directly checked into our repository.

To simplify it, this changes the shell tarball to be based on
pip installing the pypi package. Specifically, the new directory
structure for an unpack shell tarball is:
impala-shell-4.5.0-SNAPSHOT/
  impala-shell
  install_py${PYTHON_VERSION}/
  install_py${ANOTHER_PYTHON_VERSION}/
For example, install_py2.7 is the Python 2.7 pip install of impala-shell.
install_py3.8 is a Python 3.8 pip install of impala-shell. This means
that the impala-shell script simply picks the install for the
specified version of python and uses that pip install directory.
To make this more consistent across different Linux distributions, this
upgrades pip in the virtualenv to the latest.

With this, ext-py and pkg_resources.py can be removed.

This requires rearranging the shell build code. Specifically, this splits
out the code that generates impala_build_version.py so that it can run
before generating the pypi package. The shell tarball now has a dependency
on the pypi package and must run after it.

This builds on Michael Smith's work from IMPALA-11399.

Testing:
 - Ran shell tests locally
 - Built on Centos 7, Redhat 8 & 9, Ubuntu 20 & 22, SLES 15

Change-Id: Ifbb66ab2c5bc7180221f98d9bf5e38d62f4ac036
Reviewed-on: http://gerrit.cloudera.org:8080/20171
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>


> Download shell dependencies from PyPI
> -------------------------------------
>
>                 Key: IMPALA-11399
>                 URL: https://issues.apache.org/jira/browse/IMPALA-11399
>             Project: IMPALA
>          Issue Type: Improvement
>            Reporter: Michael Smith
>            Assignee: Michael Smith
>            Priority: Minor
>
> Impala's build process populates infra/python/deps from several requirements 
> files in the directory by downloading them from PyPI. shell/ext-py instead 
> checks library source code into our repository, which makes it more work to 
> upgrade to newer versions and seems somewhat redundant; it would also make it 
> easier to patch libraries in a way that may be difficult to maintain, and 
> requires updating dependency versions in 2 places (requirements.txt and the 
> local checkin).
> We could update shell to download its dependencies from PyPI as well based on 
> requirements.txt.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to