Skip to content

Conversation

@maheshrajus
Copy link
Contributor

@maheshrajus maheshrajus commented Apr 17, 2025

TEZ-4598: JDK-17: Migrate Tez to jdk17 (compile & runtime)

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/1/console in case of problems.

@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

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

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/8/console in case of problems.

@tez-yetus

This comment was marked as outdated.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/9/console in case of problems.

@tez-yetus

This comment was marked as outdated.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/10/console in case of problems.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/11/console in case of problems.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/12/console in case of problems.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/13/console in case of problems.

@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

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/24/console in case of problems.

@maheshrajus
Copy link
Contributor Author

@ayushtkn @abstractdog I've addressed all your comments and pushed the latest commit. Can you please check and approve the PR. Thank you !

@tez-yetus

This comment was marked as outdated.

@maheshrajus maheshrajus force-pushed the tezJDK17_1 branch 2 times, most recently from 9758222 to 660c5f8 Compare May 8, 2025 15:41
@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/26/console in case of problems.

@tez-yetus

This comment was marked as outdated.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/27/console in case of problems.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/28/console in case of problems.

@tez-yetus

This comment was marked as outdated.

Copy link
Member

@ayushtkn ayushtkn 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
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/29/console in case of problems.

@tez-yetus

This comment was marked as outdated.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/30/console in case of problems.

@tez-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/console in case of problems.

@tez-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 12m 12s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+0 🆗 detsecrets 0m 1s detect-secrets was not available.
+0 🆗 shelldocs 0m 1s 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 appears to include 3 new or modified test files.
_ master Compile Tests _
+0 🆗 mvndep 2m 42s Maven dependency ordering for branch
-1 ❌ mvninstall 6m 33s /branch-mvninstall-root.txt root in master failed.
+1 💚 compile 3m 6s master passed
+1 💚 checkstyle 1m 59s master passed
+1 💚 javadoc 2m 28s master passed
-1 ❌ spotbugs 0m 54s /branch-spotbugs-tez-api.txt tez-api in master failed.
-1 ❌ spotbugs 0m 26s /branch-spotbugs-tez-runtime-library.txt tez-runtime-library in master failed.
-1 ❌ spotbugs 0m 28s /branch-spotbugs-tez-dag.txt tez-dag in master failed.
-1 ❌ spotbugs 0m 22s /branch-spotbugs-tez-tests.txt tez-tests in master failed.
-1 ❌ spotbugs 0m 22s /branch-spotbugs-tez-ext-service-tests.txt tez-ext-service-tests in master failed.
-1 ❌ spotbugs 0m 18s /branch-spotbugs-root.txt root in master failed.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 12s Maven dependency ordering for patch
+1 💚 mvninstall 4m 14s the patch passed
+1 💚 codespell 0m 25s The patch generated 0 new + 7 unchanged - 1 fixed = 7 total (was 8)
+1 💚 compile 3m 0s the patch passed
+1 💚 javac 3m 0s the patch passed
+1 💚 blanks 0m 1s The patch has no blanks issues.
+1 💚 checkstyle 0m 15s The patch passed checkstyle in tez-api
+1 💚 checkstyle 0m 17s tez-runtime-library: The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚 checkstyle 0m 16s The patch passed checkstyle in tez-dag
+1 💚 checkstyle 0m 12s The patch passed checkstyle in tez-tests
+1 💚 checkstyle 0m 13s The patch passed checkstyle in tez-ext-service-tests
+1 💚 checkstyle 0m 28s root: The patch generated 0 new + 67 unchanged - 1 fixed = 67 total (was 68)
+1 💚 hadolint 0m 2s The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 yamllint 0m 1s No new issues.
+1 💚 javadoc 2m 12s the patch passed
+1 💚 spotbugs 8m 40s the patch passed
_ Other Tests _
+1 💚 unit 2m 0s tez-api in the patch passed.
+1 💚 unit 4m 30s tez-runtime-library in the patch passed.
+1 💚 unit 4m 1s tez-dag in the patch passed.
+1 💚 unit 32m 56s tez-tests in the patch passed.
+1 💚 unit 3m 48s tez-ext-service-tests in the patch passed.
+1 💚 unit 59m 12s root in the patch passed.
+1 💚 asflicense 1m 52s The patch does not generate ASF License warnings.
163m 2s
Subsystem Report/Notes
Docker ClientAPI=1.49 ServerAPI=1.49 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/artifact/out/Dockerfile
GITHUB PR #407
Optional Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs yamllint javac javadoc unit xmllint compile spotbugs checkstyle
uname Linux 9e121385f663 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-407/src/.yetus/personality.sh
git revision master / 5e7030f
Default Java Ubuntu-17.0.15+6-Ubuntu-0ubuntu122.04
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/testReport/
Max. process+thread count 2125 (vs. ulimit of 5500)
modules C: tez-api tez-runtime-library tez-dag tez-tests tez-ext-service-tests . U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/31/console
versions git=2.34.1 maven=3.6.3 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
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/console in case of problems.

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 pending tests

@tez-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 11s 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 1s 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 appears to include 3 new or modified test files.
_ master Compile Tests _
+0 🆗 mvndep 2m 30s Maven dependency ordering for branch
+1 💚 mvninstall 7m 47s master passed
+1 💚 compile 2m 59s master passed
+1 💚 checkstyle 2m 5s master passed
+1 💚 javadoc 2m 21s master passed
-1 ❌ spotbugs 0m 53s /branch-spotbugs-tez-api.txt tez-api in master failed.
-1 ❌ spotbugs 0m 24s /branch-spotbugs-tez-runtime-library.txt tez-runtime-library in master failed.
-1 ❌ spotbugs 0m 25s /branch-spotbugs-tez-dag.txt tez-dag in master failed.
-1 ❌ spotbugs 0m 21s /branch-spotbugs-tez-tests.txt tez-tests in master failed.
-1 ❌ spotbugs 0m 23s /branch-spotbugs-tez-ext-service-tests.txt tez-ext-service-tests in master failed.
-1 ❌ spotbugs 0m 17s /branch-spotbugs-root.txt root in master failed.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 11s Maven dependency ordering for patch
+1 💚 mvninstall 3m 33s the patch passed
+1 💚 codespell 0m 28s The patch generated 0 new + 7 unchanged - 1 fixed = 7 total (was 8)
+1 💚 compile 3m 6s the patch passed
+1 💚 javac 3m 6s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 0m 16s The patch passed checkstyle in tez-api
+1 💚 checkstyle 0m 16s tez-runtime-library: The patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)
+1 💚 checkstyle 0m 17s The patch passed checkstyle in tez-dag
+1 💚 checkstyle 0m 12s The patch passed checkstyle in tez-tests
+1 💚 checkstyle 0m 13s The patch passed checkstyle in tez-ext-service-tests
+1 💚 checkstyle 0m 28s root: The patch generated 0 new + 68 unchanged - 1 fixed = 68 total (was 69)
+1 💚 hadolint 0m 1s The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 yamllint 0m 0s No new issues.
+1 💚 javadoc 2m 16s the patch passed
+1 💚 spotbugs 8m 33s the patch passed
_ Other Tests _
+1 💚 unit 1m 59s tez-api in the patch passed.
+1 💚 unit 4m 18s tez-runtime-library in the patch passed.
+1 💚 unit 3m 57s tez-dag in the patch passed.
+1 💚 unit 33m 27s tez-tests in the patch passed.
+1 💚 unit 3m 45s tez-ext-service-tests in the patch passed.
-1 ❌ unit 58m 13s /patch-unit-root.txt root in the patch passed.
+1 💚 asflicense 1m 51s The patch does not generate ASF License warnings.
150m 27s
Reason Tests
Failed junit tests tez.analyzer.TestAnalyzer
Subsystem Report/Notes
Docker ClientAPI=1.49 ServerAPI=1.49 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/artifact/out/Dockerfile
GITHUB PR #407
Optional Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs yamllint javac javadoc unit xmllint compile spotbugs checkstyle
uname Linux a6e31c5af67e 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-407/src/.yetus/personality.sh
git revision master / 1369989
Default Java Ubuntu-17.0.15+6-Ubuntu-0ubuntu122.04
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/testReport/
Max. process+thread count 2125 (vs. ulimit of 5500)
modules C: tez-api tez-runtime-library tez-dag tez-tests tez-ext-service-tests . U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-407/32/console
versions git=2.34.1 maven=3.6.3 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.

@abstractdog abstractdog merged commit 51ad671 into apache:master May 13, 2025
2 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