Skip to content

TEZ-4704: Upgrade restrict-imports.enforcer to 3.0.0 to fix warning duing maven build - #476

Merged
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4704
Apr 13, 2026
Merged

TEZ-4704: Upgrade restrict-imports.enforcer to 3.0.0 to fix warning duing maven build#476
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4704

Conversation

@Aggarwal-Raghav

@Aggarwal-RaghavAggarwal-Raghav commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Check TEZ-4704,

[INFO] --- enforcer:3.6.2:enforce (banned-illegal-imports) @ tez-api ---
[WARNING] ruleName restrictImports with implementation org.apache.maven.plugins.enforcer.RestrictImportsuses the deprecated Maven Enforcer Plugin API. This will not be supported in a future version of the plugin. Please contact the rule maintainer to upgrade the rule implementation to the current API.
[INFO] Rule 0: org.apache.maven.plugins.enforcer.RestrictImports passed 

After this PR:

[INFO] --- enforcer:3.6.2:enforce (banned-illegal-imports) @ tez-api ---
[INFO] Rule 0: org.apache.maven.plugins.enforcer.RestrictImports passed

From release notes, the plugin tags we are using have no breaking change in them.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 14sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets 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 💚mvninstall4m 45smaster passed
+1 💚compile1m 49smaster passed
+1 💚javadoc1m 29smaster passed
_ Patch Compile Tests _
+1 💚mvninstall3m 42sthe patch passed
+1 💚codespell1m 35sNo new issues.
+1 💚compile1m 46sthe patch passed
+1 💚javac1m 46sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚javadoc1m 28sthe patch passed
_ Other Tests _
-1 ❌unit70m 42s/patch-unit-root.txtroot in the patch passed.
+1 💚asflicense0m 55sThe patch does not generate ASF License warnings.
89m 52s
ReasonTests
Failed junit teststez.dag.history.recovery.TestRecoveryService
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-476/1/artifact/out/Dockerfile
GITHUB PR#476
Optional Testsdupname asflicense javac javadoc unit codespell detsecrets xmllint compile
unameLinux 9bf6c245f009 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-476/src/.yetus/personality.sh
git revisionmaster / e41e929
Default JavaUbuntu-21.0.10+7-Ubuntu-124.04
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-476/1/testReport/
Max. process+thread count1429 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-476/1/console
versionsgit=2.43.0 maven=3.9.14 codespell=2.4.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
Contributor

TestRecoveryService is a known flakiness, I don't insist on re-running tests

@abstractdog
abstractdog self-requested a review April 13, 2026 12:08

@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

@abstractdog
abstractdog merged commit 2ca44b4 into apache:masterApr 13, 2026
4 checks passed
@Aggarwal-Raghav
Aggarwal-Raghav deleted the TEZ-4704 branch May 1, 2026 05:52
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.

3 participants

@Aggarwal-Raghav@tez-yetus@abstractdog