Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/20172

to look at the new patch set (#3).

Change subject: PROTOTYPE: Rearrange shell directory structure to match the 
pypi package
......................................................................

PROTOTYPE: Rearrange shell directory structure to match the pypi package

Impala shell's directory structure does not match what the
pypi package needs, so it currently requires the make_python_package.sh
script to create the package with the appropriate structure.
This rearranges the directory structure so that it is exactly what
we need for the pypi package. Some files are still generated by CMake
(e.g. the impala version python file and the thrift generated python).
After the files are generated, the code can simply use setup.py sdist
to produce the appropriate archive.

This moves the shell tarball packaging code to the packaging directory
where the pypi code used to live.

As part of this, the Thrift-generated python code no longer lives in
gen-py. It is placed directly in the shell directory. This fixes up
the paths so that other Python code is referring to the shell code
as it would be if it were pip installed.

Change-Id: I1b54fa90f48c6fe20c66a01403e146a941fc5d4f
---
M README-build.md
M bin/impala-shell.sh
M bin/rat_exclude_files.txt
M bin/set-pythonpath.sh
M common/thrift/CMakeLists.txt
M shell/.gitignore
M shell/CMakeLists.txt
R shell/MANIFEST.in
R shell/README.md
R shell/packaging/impala-shell
D shell/packaging/make_python_package.sh
R shell/packaging/make_shell_tarball.sh
R shell/requirements.txt
R shell/setup.py
M tests/custom_cluster/test_hs2_fault_injection.py
M tests/custom_cluster/test_shell_interactive_reconnect.py
M tests/custom_cluster/test_thrift_socket.py
M tests/shell/test_cookie_util.py
M tests/shell/test_shell_client.py
M tests/shell/test_shell_commandline.py
M tests/shell/test_shell_interactive.py
M tests/shell/util.py
22 files changed, 31 insertions(+), 106 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/72/20172/3
--
To view, visit http://gerrit.cloudera.org:8080/20172
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1b54fa90f48c6fe20c66a01403e146a941fc5d4f
Gerrit-Change-Number: 20172
Gerrit-PatchSet: 3
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>

Reply via email to