stevenschlansker commented on code in PR #2256:
URL: https://github.com/apache/fury/pull/2256#discussion_r2110230704
##########
ci/run_ci.sh:
##########
@@ -146,10 +146,11 @@ integration_tests() {
jdk17_plus_tests() {
java -version
+ export
JDK_JAVA_OPTIONS="--add-opens=java.base/java.nio=org.apache.arrow.memory.core,ALL-UNNAMED"
echo "Executing fury java tests"
cd "$ROOT/java"
set +e
- mvn -T10 --batch-mode --no-transfer-progress test install -pl
'!fury-format,!fury-testsuite'
+ mvn -T10 --batch-mode --no-transfer-progress install
Review Comment:
Is there a reason fury-format is excluded here? I think it means integration
tests can run with the wrong fury-format jar - one from a snapshot repo,
instead of the one from the PR build
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
