Skip to content

TEZ-4670: Upgrade spotless plugin to 3.x - #447

Merged
abstractdog merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4670
Jan 13, 2026
Merged

TEZ-4670: Upgrade spotless plugin to 3.x#447
abstractdog merged 1 commit into
apache:masterfrom
abstractdog:TEZ-4670

Conversation

@abstractdog

Copy link
Copy Markdown
Contributor

No description provided.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec29m 30sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo 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 💚mvninstall11m 25smaster passed
+1 💚compile2m 29smaster passed
+1 💚javadoc1m 41smaster passed
_ Patch Compile Tests _
+1 💚mvninstall5m 6sthe patch passed
+1 💚codespell1m 3sNo new issues.
+1 💚compile2m 33sthe patch passed
+1 💚javac2m 33sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚javadoc1m 21sthe patch passed
_ Other Tests _
+1 💚unit71m 27sroot in the patch passed.
+1 💚asflicense0m 37sThe patch does not generate ASF License warnings.
129m 1s
SubsystemReport/Notes
DockerClientAPI=1.52 ServerAPI=1.52 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-447/1/artifact/out/Dockerfile
GITHUB PR#447
Optional Testsdupname asflicense javac javadoc unit codespell detsecrets xmllint compile
unameLinux f64b572c6698 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-agent/workspace/tez-multibranch_PR-447/src/.yetus/personality.sh
git revisionmaster / 78c3e12
Default JavaUbuntu-21.0.9+10-Ubuntu-124.04
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-447/1/testReport/
Max. process+thread count2096 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-447/1/console
versionsgit=2.43.0 maven=3.8.7 codespell=2.0.0
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@ayushtkn

Copy link
Copy Markdown
Member

@abstractdog I think we need to touch some files to test, I don't think the CI ran the spotbugs here

@abstractdog

Copy link
Copy Markdown
ContributorAuthor

@abstractdog I think we need to touch some files to test, I don't think the CI ran the spotbugs here

be mindful that it's about the spotless plugin, not spotbugs
spotless already kicks in for a simple mvn clean install

@ayushtknayushtkn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahhh, my bad
LGTM

@abstractdog
abstractdog merged commit 533b013 into apache:masterJan 13, 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.

3 participants

@abstractdog@tez-yetus@ayushtkn