Skip to content

TEZ-4726: Replace deprecated java.net.URL constructors with URI alternatives#508

Open
maheshrajus wants to merge 1 commit into
apache:masterfrom
maheshrajus:TEZ-4726
Open

TEZ-4726: Replace deprecated java.net.URL constructors with URI alternatives#508
maheshrajus wants to merge 1 commit into
apache:masterfrom
maheshrajus:TEZ-4726

Conversation

@maheshrajus

Copy link
Copy Markdown
Contributor

What changes were proposed in this PR?

Replace deprecated java.net.URL constructors with URI-based alternatives for Java 20+ compatibility

The java.net.URL constructors have been deprecated
since Java 20 and are marked for removal in a future JDK release.
This patch replaces all usages of deprecated URL constructors across the
codebase with the recommended URI-based alternatives.

Linked JIRA:
https://issues.apache.org/jira/browse/TEZ-4726

How was this patch tested?

  • with existed UT

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 2m 23s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 7 new or modified test files.
_ master Compile Tests _
+0 🆗 mvndep 0m 26s Maven dependency ordering for branch
+1 💚 mvninstall 3m 1s master passed
+1 💚 compile 2m 38s master passed
+1 💚 checkstyle 2m 1s master passed
+1 💚 javadoc 1m 58s master passed
+0 🆗 spotbugs 0m 51s tez-api in master has 92 extant spotbugs warnings.
+0 🆗 spotbugs 0m 39s tez-runtime-library in master has 143 extant spotbugs warnings.
+0 🆗 spotbugs 0m 52s tez-dag in master has 537 extant spotbugs warnings.
+0 🆗 spotbugs 0m 30s tez-tests in master has 6 extant spotbugs warnings.
+0 🆗 spotbugs 0m 30s tez-plugins/tez-history-parser in master has 21 extant spotbugs warnings.
+0 🆗 spotbugs 0m 24s tez-plugins/tez-aux-services in master has 5 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 6s Maven dependency ordering for patch
+1 💚 mvninstall 2m 25s the patch passed
+1 💚 codespell 0m 56s No new issues.
+1 💚 compile 2m 32s the patch passed
+1 💚 javac 2m 32s root generated 0 new + 711 unchanged - 31 fixed = 711 total (was 742)
+1 💚 blanks 0m 0s The patch has no blanks issues.
-0 ⚠️ checkstyle 0m 17s /results-checkstyle-tez-plugins_tez-aux-services.txt tez-plugins/tez-aux-services: The patch generated 2 new + 82 unchanged - 2 fixed = 84 total (was 84)
+1 💚 javadoc 2m 30s the patch passed
+1 💚 spotbugs 5m 34s the patch passed
_ Other Tests _
+1 💚 unit 60m 44s root in the patch passed.
+1 💚 asflicense 2m 19s The patch does not generate ASF License warnings.
98m 39s
Subsystem Report/Notes
Docker ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-508/1/artifact/out/Dockerfile
Optional Tests dupname compile unit asflicense javac javadoc spotbugs checkstyle codespell detsecrets
uname Linux a4c56bb2b7b7 5.15.0-179-generic #189-Ubuntu SMP Tue May 5 18:20:56 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality tez-personality.sh
git revision master / cba5689
Default Java Eclipse Adoptium-21.0.11+10-LTS
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-508/1/testReport/
Max. process+thread count 1375 (vs. ulimit of 5500)
modules C: tez-api tez-runtime-library tez-dag tez-tests tez-ext-service-tests tez-plugins/tez-history-parser tez-plugins/tez-aux-services U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-508/1/console
versions git=2.43.0 maven=3.9.15 spotbugs=4.9.3 codespell=2.4.1
Powered by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@maheshrajus

Copy link
Copy Markdown
Contributor Author

@abstractdog @Aggarwal-Raghav
Could you please take a look and approve it when you have a moment? Thank you !

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.

2 participants