Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13544
Change subject: Fix integration of kudu-hive.jar ...................................................................... Fix integration of kudu-hive.jar IMPALA-8503 added downloading kudu-hive.jar and adding it to HADOOP_CLASSPATH in run-hive-server.sh to allow the Hive Metastore to start with Kudu's HMS plugin. There are two problems with this that are fixed by this patch: - Previously, we fully specify the expected jar filename based on the value of IMPALA_KUDU_JAVA_VERSION when adding it to HADOOP_CLASSPATH but this is overly restrictive for users who may wish to override this value in impala-config-branch.sh to build their own branch with a different version of the kudu-hive.jar This patch relaxes this restriction by adding any jar containing the string kudu-hive in IMPALA_KUDU_JAVA_HOME to HADOOP_CLASSPATH - In bootstrap_toolchain, we don't download a package if its directory already exists. Since the 'kudu' and 'kudu-java' packages download to the same directory, this led to a race condition where 'kudu-java' might not be downloaded if 'kudu' had already been unpacked when it started. This patch fixes this by inspecting the contents of the Kudu package directory to look for specific files expected for each Kudu package. Change-Id: Ia874efa63d9114716da3b2ac8921e78b518bdae5 --- M bin/bootstrap_toolchain.py M bin/impala-config.sh M testdata/bin/run-hive-server.sh 3 files changed, 21 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/44/13544/1 -- To view, visit http://gerrit.cloudera.org:8080/13544 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ia874efa63d9114716da3b2ac8921e78b518bdae5 Gerrit-Change-Number: 13544 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Marshall <[email protected]>
