kszucs commented on a change in pull request #10300:
URL: https://github.com/apache/arrow/pull/10300#discussion_r631815612
##########
File path: dev/tasks/jars/build-java.sh
##########
@@ -23,12 +23,14 @@ CPP_BUILD_DIR=$GITHUB_WORKSPACE/arrow/dist/
pushd java
# build the entire project
- mvn clean install -q -DskipTests -P arrow-jni
-Darrow.cpp.build.dir=$CPP_BUILD_DIR
- # test only gandiva
- mvn test -q -P arrow-jni -pl gandiva -Dgandiva.cpp.build.dir=$CPP_BUILD_DIR
+ mvn clean install -DskipTests -P arrow-jni
-Darrow.cpp.build.dir=$CPP_BUILD_DIR
Review comment:
I think we can move this file to the `ci/scripts` directory as well
using the same prefix as the cpp build scripts.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
