Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24456 )
Change subject: IMPALA-15100: Stop building Thrift's python library ...................................................................... IMPALA-15100: Stop building Thrift's python library Impala gets Thrift's python library via PyPi and no longer needs to build it in the toolchain. This removes Thrift's Python build but preserves the code that includes fb303.thrift in the archive. Since this was the only use for Python 2, this removes the Python 2 build. This also does misc cleanup to use scopes for environment variables rather than manual unset calls. Change-Id: If7ba893da7b7eeaa4f31292e4526f8dded9d3934 Reviewed-on: http://gerrit.cloudera.org:8080/24456 Reviewed-by: Michael Smith <[email protected]> Reviewed-by: Laszlo Gaal <[email protected]> Tested-by: Joe McDonnell <[email protected]> --- M buildall.sh M source/thrift/build.sh 2 files changed, 39 insertions(+), 61 deletions(-) Approvals: Michael Smith: Looks good to me, but someone else must approve Laszlo Gaal: Looks good to me, approved Joe McDonnell: Verified -- To view, visit http://gerrit.cloudera.org:8080/24456 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If7ba893da7b7eeaa4f31292e4526f8dded9d3934 Gerrit-Change-Number: 24456 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
