Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10361 )
Change subject: Remove IMPALA_THRIFT_JAVA_VERSION and untested Darwin Thrift versions. ...................................................................... Remove IMPALA_THRIFT_JAVA_VERSION and untested Darwin Thrift versions. The singular use of IMPALA_THRIFT_JAVA_VERSION was in impala-parent/pom.xml. We can reduce complexity by just inlining the version there, like we do with several other Java dependencies. Meanwhile, with the upgrade to Thrift 0.9.3, it doesn't make sense to retain the Darwin path for Thrift 0.9.2. The reality is likely that nobody is building Impala on Darwin. Cherry-picks: not for 2.x Change-Id: I98f8c0b344afd001864653659c045b5d3c3e94ac Reviewed-on: http://gerrit.cloudera.org:8080/10361 Reviewed-by: Tianyi Wang <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M bin/impala-config.sh M impala-parent/pom.xml 2 files changed, 1 insertion(+), 7 deletions(-) Approvals: Tianyi Wang: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10361 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I98f8c0b344afd001864653659c045b5d3c3e94ac Gerrit-Change-Number: 10361 Gerrit-PatchSet: 3 Gerrit-Owner: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Tianyi Wang <[email protected]>
