You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Joe McDonnell (Code Review)" <ge...@cloudera.org> on 2020/05/29 01:05:16 UTC
[Impala-ASF-CR] IMPALA-9760: Add IMPALA TOOLCHAIN PACKAGES HOME to prepare for GCC7
Hello Tim Armstrong, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15991
to look at the new patch set (#3).
Change subject: IMPALA-9760: Add IMPALA_TOOLCHAIN_PACKAGES_HOME to prepare for GCC7
......................................................................
IMPALA-9760: Add IMPALA_TOOLCHAIN_PACKAGES_HOME to prepare for GCC7
The locations for native-toolchain packages in IMPALA_TOOLCHAIN
currently do not include the compiler version. This means that
the toolchain can't distinguish between native-toolchain packages
built with gcc 4.9.2 versus gcc 7.5.0. The collisions can cause
issues when switching back and forth between branches.
This introduces the IMPALA_TOOLCHAIN_PACKAGES_HOME environment
variable, which is a location inside IMPALA_TOOLCHAIN that would
hold native-toolchain packages. Currently, it is set to the same
as IMPALA_TOOLCHAIN, so there is no difference in behavior.
This lays the groundwork to add the compiler version to this
path when switching to GCC7.
Testing:
- Built with IMPALA_TOOLCHAIN_PACKAGES_HOME=$IMPALA_TOOLCHAIN/test
and verified that it will work with a path other than
$IMPALA_TOOLCHAIN.
- Core tests
Change-Id: I1ff641e503b2161baf415355452f86b6c8bfb15b
---
M CMakeLists.txt
M be/CMakeLists.txt
M be/src/service/CMakeLists.txt
M bin/bootstrap_toolchain.py
M bin/distcc/distcc.sh
M bin/dump_breakpad_symbols.py
M bin/impala-config.sh
M bin/impala-shell.sh
M bin/jenkins/finalize.sh
M bin/run-backend-tests.sh
M bin/run-binary.sh
M bin/run-jvm-binary.sh
M bin/run_clang_tidy.sh
M bin/set-ld-library-path.sh
M bin/set-pythonpath.sh
M cmake_modules/clang_toolchain.cmake
M cmake_modules/toolchain.cmake
M docker/setup_build_context.py
M fe/pom.xml
M infra/python/bootstrap_virtualenv.py
M shell/make_shell_tarball.sh
M shell/packaging/make_python_package.sh
M testdata/datasets/tpcds/preload
M testdata/datasets/tpch/preload
24 files changed, 101 insertions(+), 78 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/91/15991/3
--
To view, visit http://gerrit.cloudera.org:8080/15991
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1ff641e503b2161baf415355452f86b6c8bfb15b
Gerrit-Change-Number: 15991
Gerrit-PatchSet: 3
Gerrit-Owner: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>