Attila Jeges has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15072 )

Change subject: IMPALA-9265: Support for toolchain Kudu to provide Java 
artifacts
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15072/3/source/kudu/build.sh
File source/kudu/build.sh:

http://gerrit.cloudera.org:8080/#/c/15072/3/source/kudu/build.sh@137
PS3, Line 137:   local JAVA_INSTALL_DIR="$LOCAL_INSTALL/java"
             :   mkdir -p "$JAVA_INSTALL_DIR"
             :   pushd java
             :   export GRADLE_USER_HOME="$(pwd)"
             :   wrap ./gradlew :kudu-hive:assemble :kudu-client:assemble
             :   # Copy kudu-hive jars to JAVA_INSTALL_DIR.
             :   local F
             :   for F in kudu-hive/build/libs/kudu-hive-*.jar; do
             :     cp "$F" "$JAVA_INSTALL_DIR"
             :   done
             :   # Install kudu-client artifacts to the Local Maven Repository:
             :   wrap ./gradlew 
-Dmaven.repo.local="${JAVA_INSTALL_DIR}/repository" :kudu-client:install
             :   popd
I've also tested Impala quickly with kudu-hive and kudu-client built like this 
locally on my dev machine.

It looks good, impala builds without an issue and the kudu-related E2E tests 
that I've tried passed.



--
To view, visit http://gerrit.cloudera.org:8080/15072
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iba03dfe9c302513b825cbed7146c582e7d97c3af
Gerrit-Change-Number: 15072
Gerrit-PatchSet: 3
Gerrit-Owner: Attila Jeges <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Comment-Date: Mon, 27 Jan 2020 14:38:17 +0000
Gerrit-HasComments: Yes

Reply via email to