Skip to content

Conversation

@ayushtkn
Copy link
Member

@ayushtkn ayushtkn commented Jun 17, 2025

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

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 13m 28s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+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 8m 39s master passed
+1 💚 compile 1m 16s master passed
+1 💚 javadoc 1m 3s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 2m 32s the patch passed
+1 💚 codespell 0m 26s No new issues.
+1 💚 compile 1m 14s the patch passed
+1 💚 javac 1m 14s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
-1 ❌ hadolint 0m 1s /results-hadolint.txt The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 yamllint 0m 0s No new issues.
+1 💚 javadoc 0m 44s the patch passed
_ Other Tests _
+1 💚 unit 59m 0s root in the patch passed.
+1 💚 asflicense 0m 28s The patch does not generate ASF License warnings.
89m 45s
Subsystem Report/Notes
Docker ClientAPI=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 Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs yamllint javac javadoc unit xmllint compile
uname Linux 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 tool maven
Personality /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-418/src/.yetus/personality.sh
git revision master / 835f3ca
Default Java Ubuntu-21.0.7+6-Ubuntu-0ubuntu124.04
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/4/testReport/
Max. process+thread count 2110 (vs. ulimit of 5500)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/4/console
versions git=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 by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

This comment was marked as outdated.

@ayushtkn ayushtkn changed the title WIP TEZ-4636: Move to JDK-21+ Jun 17, 2025
@tez-yetus

This comment was marked as outdated.

@tez-yetus

This comment was marked as outdated.

@ayushtkn ayushtkn requested a review from abstractdog June 17, 2025 15:49
Copy link
Contributor

@maheshrajus maheshrajus left a comment

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

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 19s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+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 11m 3s master passed
+1 💚 compile 2m 27s master passed
+1 💚 javadoc 1m 32s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 4m 51s the patch passed
+1 💚 codespell 0m 59s No new issues.
+1 💚 compile 2m 24s the patch passed
+1 💚 javac 2m 24s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 hadolint 0m 1s No new issues.
+1 💚 shellcheck 0m 1s No new issues.
+1 💚 yamllint 0m 1s No new issues.
+1 💚 javadoc 1m 11s the patch passed
_ Other Tests _
-1 ❌ unit 68m 43s /patch-unit-root.txt root in the patch passed.
+1 💚 asflicense 0m 36s The patch does not generate ASF License warnings.
95m 40s
Reason Tests
Failed junit tests tez.test.TestRecovery
Subsystem Report/Notes
Docker ClientAPI=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 Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs yamllint javac javadoc unit xmllint compile
uname Linux 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 tool maven
Personality /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-418/src/.yetus/personality.sh
git revision master / 3f880c9
Default Java Ubuntu-21.0.7+6-Ubuntu-0ubuntu124.04
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/7/testReport/
Max. process+thread count 2109 (vs. ulimit of 5500)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-418/7/console
versions git=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 by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@abstractdog abstractdog left a comment

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:master Jun 20, 2025
4 checks passed
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.

4 participants