Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13542 )
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: I4ac79c3e9b8625ba54145dba23c69fd5117f35c7 Reviewed-on: http://gerrit.cloudera.org:8080/13542 Reviewed-by: Thomas Marshall <[email protected]> Reviewed-by: Hao Hao <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M bin/bootstrap_toolchain.py M bin/impala-config.sh M testdata/bin/run-hive-server.sh 3 files changed, 21 insertions(+), 5 deletions(-) Approvals: Thomas Marshall: Looks good to me, approved Hao Hao: Looks good to me, but someone else must approve Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/13542 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4ac79c3e9b8625ba54145dba23c69fd5117f35c7 Gerrit-Change-Number: 13542 Gerrit-PatchSet: 5 Gerrit-Owner: Thomas Marshall <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Thomas Marshall <[email protected]>
