Skip to content

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

Merged
abstractdog merged 1 commit into
apache:masterfrom
maheshrajus:TEZ-4726
Jun 16, 2026
Merged

TEZ-4726: Replace deprecated java.net.URL constructors with URI alternatives#508
abstractdog merged 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

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec2m 23sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 7 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep0m 26sMaven dependency ordering for branch
+1 💚mvninstall3m 1smaster passed
+1 💚compile2m 38smaster passed
+1 💚checkstyle2m 1smaster passed
+1 💚javadoc1m 58smaster passed
+0 🆗spotbugs0m 51stez-api in master has 92 extant spotbugs warnings.
+0 🆗spotbugs0m 39stez-runtime-library in master has 143 extant spotbugs warnings.
+0 🆗spotbugs0m 52stez-dag in master has 537 extant spotbugs warnings.
+0 🆗spotbugs0m 30stez-tests in master has 6 extant spotbugs warnings.
+0 🆗spotbugs0m 30stez-plugins/tez-history-parser in master has 21 extant spotbugs warnings.
+0 🆗spotbugs0m 24stez-plugins/tez-aux-services in master has 5 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 6sMaven dependency ordering for patch
+1 💚mvninstall2m 25sthe patch passed
+1 💚codespell0m 56sNo new issues.
+1 💚compile2m 32sthe patch passed
+1 💚javac2m 32sroot generated 0 new + 711 unchanged - 31 fixed = 711 total (was 742)
+1 💚blanks0m 0sThe patch has no blanks issues.
-0 ⚠️checkstyle0m 17s/results-checkstyle-tez-plugins_tez-aux-services.txttez-plugins/tez-aux-services: The patch generated 2 new + 82 unchanged - 2 fixed = 84 total (was 84)
+1 💚javadoc2m 30sthe patch passed
+1 💚spotbugs5m 34sthe patch passed
_ Other Tests _
+1 💚unit60m 44sroot in the patch passed.
+1 💚asflicense2m 19sThe patch does not generate ASF License warnings.
98m 39s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-508/1/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc spotbugs checkstyle codespell detsecrets
unameLinux 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 toolmaven
Personalitytez-personality.sh
git revisionmaster / cba5689
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-508/1/testReport/
Max. process+thread count1375 (vs. ulimit of 5500)
modulesC: 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 outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-508/1/console
versionsgit=2.43.0 maven=3.9.15 spotbugs=4.9.3 codespell=2.4.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@maheshrajus

Copy link
Copy Markdown
ContributorAuthor

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

@abstractdog
abstractdog self-requested a review June 11, 2026 21:04

@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.

LGTM, replaced all the occurrences

@maheshrajus

Copy link
Copy Markdown
ContributorAuthor

@abstractdog@Aggarwal-Raghav
Can we merge this PR? Currently, there are no open review comments and all checks are green. Thanks!

@abstractdog
abstractdog merged commit 92dedd5 into apache:masterJun 16, 2026
4 checks passed
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

@maheshrajus@tez-yetus@abstractdog@Aggarwal-Raghav