Skip to content

Conversation

@maheshrajus
Copy link
Contributor

adding profile for jdk17 builds

@tez-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 13m 37s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
-1 ❌ mvninstall 7m 30s root in master failed.
-1 ❌ compile 0m 39s root in master failed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04.
+1 💚 compile 1m 30s master passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚 javadoc 1m 3s master passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚 javadoc 0m 46s master passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
_ Patch Compile Tests _
+1 💚 mvninstall 2m 58s the patch passed
+1 💚 compile 1m 23s the patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚 javac 1m 23s the patch passed
+1 💚 compile 1m 13s the patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
+1 💚 javac 1m 13s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 xml 0m 1s The patch has no ill-formed XML file.
+1 💚 javadoc 0m 45s the patch passed with JDK Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04
+1 💚 javadoc 0m 45s the patch passed with JDK Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
_ Other Tests _
+1 💚 unit 64m 23s root in the patch passed.
+1 💚 asflicense 0m 27s The patch does not generate ASF License warnings.
97m 45s
Subsystem Report/Notes
Docker ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-404/1/artifact/out/Dockerfile
GITHUB PR #404
JIRA Issue TEZ-4618
Optional Tests dupname asflicense javac javadoc unit xml compile
uname Linux 8fb7e2d9d991 5.15.0-130-generic #140-Ubuntu SMP Wed Dec 18 17:59:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/tez.sh
git revision master / 7c24f25
Default Java Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
Multi-JDK versions /usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_442-8u442-b06us1-0ubuntu122.04-b06
mvninstall https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-404/1/artifact/out/branch-mvninstall-root.txt
compile https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-404/1/artifact/out/branch-compile-root-jdkUbuntu-11.0.26+4-post-Ubuntu-1ubuntu122.04.txt
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-404/1/testReport/
Max. process+thread count 1556 (vs. ulimit of 5500)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-404/1/console
versions git=2.34.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<argLine>--add-opens java.base/java.lang=ALL-UNNAMED</argLine>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

argLine should also contain the -Xmx1024m -XX:+HeapDumpOnOutOfMemoryError, otherwise it will get overwritten. This can be confirmend by running test in debug mode. Rest of the configuration, evn and sytem properties are carried forward to java17 profile from the initial maven surefire plugin

mvn test -Dtest="org.apache.tez.dag.history.recovery.TestRecoveryService#testRecoveryFlushOnSummaryEvent" -X | grep "argLine"

@maheshrajus
Copy link
Contributor Author

Raised PR.
#405

@maheshrajus maheshrajus closed this Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants