Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10361 )
Change subject: Remove IMPALA_THRIFT_JAVA_VERSION and untested Darwin Thrift versions. ...................................................................... Patch Set 1: In terms of additional color, I've heard from multiple folks that the environment variables are a major pain. It makes it hard to get started in Impala, harder to get tools set up just right, etc. One of the motivations here is that I've run into tools that parse pom.xml files but are confused by variable references. This is a drop in a big ocean, but I think it's directionally reasonable to work towards fewer things in impala-config.sh. -- 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: comment Gerrit-Change-Id: I98f8c0b344afd001864653659c045b5d3c3e94ac Gerrit-Change-Number: 10361 Gerrit-PatchSet: 1 Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Comment-Date: Thu, 10 May 2018 04:17:38 +0000 Gerrit-HasComments: No