Riza Suminto has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18636
Change subject: IMPALA-11369: Separate thrift compiler for different component ...................................................................... IMPALA-11369: Separate thrift compiler for different component Impala used to have one thrift compiler version to compile C++, Java, and Python code. However, Thrift serialization/deserialization between minor version is mostly compatible with each other. So it is possible to have different thrift compiler version for different target code. It is also beneficial to do so because different component can then be upgraded independently. This patch implement the infrastructure change required to do so. It replace most of 'THRIFT_*' environment variable anc cmake variable with 'THRFIT_CPP_*', 'THRFIT_JAVA_*', and 'THRFIT_PY_*' to compile C++, Java, and Python code accordingly. All three still refer to the same thrift version (thrift-0.11.0). Testing: - Build Impala and pass core tests. Change-Id: I56479dc69b79024d1a4d09211bbe88a61fa0c6a4 --- M CMakeLists.txt M README-build.md M be/CMakeLists.txt M bin/bootstrap_toolchain.py M bin/impala-config.sh M bin/impala-shell.sh M bin/set-pythonpath.sh D cmake_modules/FindThrift.cmake A cmake_modules/FindThriftCpp.cmake A cmake_modules/FindThriftJava.cmake A cmake_modules/FindThriftPython.cmake M common/thrift/CMakeLists.txt M shell/make_shell_tarball.sh M shell/packaging/make_python_package.sh 14 files changed, 384 insertions(+), 133 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/36/18636/1 -- To view, visit http://gerrit.cloudera.org:8080/18636 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I56479dc69b79024d1a4d09211bbe88a61fa0c6a4 Gerrit-Change-Number: 18636 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <[email protected]>
