Skip to content

TEZ-4636: Move to JDK-21+ - #418

Merged
ayushtkn merged 7 commits into
apache:masterfrom
ayushtkn:jdk21
Jun 20, 2025
Merged

TEZ-4636: Move to JDK-21+#418
ayushtkn merged 7 commits into
apache:masterfrom
ayushtkn:jdk21

Conversation

@ayushtkn

@ayushtknayushtkn commented Jun 17, 2025

Copy link
Copy Markdown
Member

Example Job with Tez-JDK-21 & Hadoop-3.4.1
image

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec13m 28sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+0 🆗xmllint0m 0sxmllint was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe 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 💚mvninstall8m 39smaster passed
+1 💚compile1m 16smaster passed
+1 💚javadoc1m 3smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 32sthe patch passed
+1 💚codespell0m 26sNo new issues.
+1 💚compile1m 14sthe patch passed
+1 💚javac1m 14sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
-1 ❌hadolint0m 1s/results-hadolint.txtThe patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 💚shellcheck0m 0sNo new issues.
+1 💚yamllint0m 0sNo new issues.
+1 💚javadoc0m 44sthe patch passed
_ Other Tests _
+1 💚unit59m 0sroot in the patch passed.
+1 💚asflicense0m 28sThe patch does not generate ASF License warnings.
89m 45s
SubsystemReport/Notes
DockerClientAPI=1.50 ServerAPI=1.50 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/4/artifact/out/Dockerfile
GITHUB PR#418
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs yamllint javac javadoc unit xmllint compile
unameLinux 530c21501a8d 5.15.0-136-generic #147-Ubuntu SMP Sat Mar 15 15:53:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-418/src/.yetus/personality.sh
git revisionmaster / 835f3ca
Default JavaUbuntu-21.0.7+6-Ubuntu-0ubuntu124.04
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/4/testReport/
Max. process+thread count2110 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/4/console
versionsgit=2.43.0 maven=3.8.7 hadolint=1.18.0-0-g76eee5c codespell=2.0.0 shellcheck=0.7.1 yamllint=1.24.2
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

This comment was marked as outdated.

@ayushtknayushtkn changed the title WIPTEZ-4636: Move to JDK-21+Jun 17, 2025
@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@maheshrajusmaheshrajus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as off-topic.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 19sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+0 🆗xmllint0m 0sxmllint was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe 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 💚mvninstall11m 3smaster passed
+1 💚compile2m 27smaster passed
+1 💚javadoc1m 32smaster passed
_ Patch Compile Tests _
+1 💚mvninstall4m 51sthe patch passed
+1 💚codespell0m 59sNo new issues.
+1 💚compile2m 24sthe patch passed
+1 💚javac2m 24sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚hadolint0m 1sNo new issues.
+1 💚shellcheck0m 1sNo new issues.
+1 💚yamllint0m 1sNo new issues.
+1 💚javadoc1m 11sthe patch passed
_ Other Tests _
-1 ❌unit68m 43s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 36sThe patch does not generate ASF License warnings.
95m 40s
ReasonTests
Failed junit teststez.test.TestRecovery
SubsystemReport/Notes
DockerClientAPI=1.50 ServerAPI=1.50 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/7/artifact/out/Dockerfile
GITHUB PR#418
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs yamllint javac javadoc unit xmllint compile
unameLinux 50fa925a24f8 5.15.0-139-generic #149-Ubuntu SMP Fri Apr 11 22:06:13 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-418/src/.yetus/personality.sh
git revisionmaster / 3f880c9
Default JavaUbuntu-21.0.7+6-Ubuntu-0ubuntu124.04
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/7/testReport/
Max. process+thread count2109 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/7/console
versionsgit=2.43.0 maven=3.8.7 hadolint=1.18.0-0-g76eee5c codespell=2.0.0 shellcheck=0.7.1 yamllint=1.24.2
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdogabstractdog left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1
TestRecovery is a known flakyness
nice work!

would you be so kind @ayushtkn to write an announcement to dev@ and user@ informing people that we're on JDK21 since 1.0.0

@ayushtkn
ayushtkn merged commit 03b5f1d into apache:masterJun 20, 2025
Sign up for freeto 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.

4 participants

@ayushtkn@tez-yetus@abstractdog@maheshrajus