Skip to content

Comments

TEZ-4687: Spotless: enforce endWithNewline#459

Open
abstractdog wants to merge 1 commit intoapache:masterfrom
abstractdog:TEZ-4687
Open

TEZ-4687: Spotless: enforce endWithNewline#459
abstractdog wants to merge 1 commit intoapache:masterfrom
abstractdog:TEZ-4687

Conversation

@abstractdog
Copy link
Contributor

No description provided.

@abstractdog abstractdog requested a review from ayushtkn February 24, 2026 13:38
@Aggarwal-Raghav
Copy link
Contributor

LGTM +1

@tez-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 5m 2s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+0 🆗 detsecrets 0m 0s detect-secrets 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 9 new or modified test files.
_ master Compile Tests _
+0 🆗 mvndep 2m 15s Maven dependency ordering for branch
+1 💚 mvninstall 7m 16s master passed
+1 💚 compile 4m 26s master passed
+1 💚 checkstyle 2m 46s master passed
+1 💚 javadoc 3m 40s master passed
+0 🆗 spotbugs 1m 27s tez-api in master has 545 extant spotbugs warnings.
+0 🆗 spotbugs 0m 29s tez-common in master has 13 extant spotbugs warnings.
+0 🆗 spotbugs 0m 34s tez-runtime-internals in master has 111 extant spotbugs warnings.
+0 🆗 spotbugs 0m 40s tez-runtime-library in master has 235 extant spotbugs warnings.
+0 🆗 spotbugs 0m 33s tez-mapreduce in master has 124 extant spotbugs warnings.
+0 🆗 spotbugs 0m 56s tez-dag in master has 749 extant spotbugs warnings.
+0 🆗 spotbugs 0m 28s tez-ext-service-tests in master has 62 extant spotbugs warnings.
+0 🆗 spotbugs 0m 28s tez-plugins/tez-history-parser in master has 21 extant spotbugs warnings.
+0 🆗 spotbugs 3m 45s root in master has 1935 extant spotbugs warnings.
+0 🆗 spotbugs 0m 28s tez-tools/analyzers/job-analyzer in master has 20 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 9s Maven dependency ordering for patch
+1 💚 mvninstall 5m 0s the patch passed
+1 💚 codespell 0m 49s No new issues.
+1 💚 compile 4m 27s the patch passed
+1 💚 javac 4m 27s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 2m 24s the patch passed
+1 💚 javadoc 3m 29s the patch passed
+1 💚 spotbugs 10m 51s the patch passed
_ Other Tests _
+1 💚 unit 2m 6s tez-api in the patch passed.
+1 💚 unit 0m 27s tez-common in the patch passed.
+1 💚 unit 0m 45s tez-runtime-internals in the patch passed.
+1 💚 unit 4m 21s tez-runtime-library in the patch passed.
+1 💚 unit 1m 6s tez-mapreduce in the patch passed.
+1 💚 unit 4m 51s tez-dag in the patch passed.
+1 💚 unit 3m 37s tez-ext-service-tests in the patch passed.
+1 💚 unit 2m 4s tez-history-parser in the patch passed.
+1 💚 unit 2m 21s job-analyzer in the patch passed.
+1 💚 unit 58m 57s root in the patch passed.
+1 💚 asflicense 2m 46s The patch does not generate ASF License warnings.
149m 57s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-459/1/artifact/out/Dockerfile
GITHUB PR #459
Optional Tests dupname asflicense javac javadoc unit codespell detsecrets xmllint compile spotbugs checkstyle
uname Linux a761b36c4bde 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 tool maven
Personality /home/jenkins/jenkins-home/workspace/tez-multibranch_PR-459/src/.yetus/personality.sh
git revision master / 9677b39
Default Java Ubuntu-21.0.10+7-Ubuntu-124.04
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-459/1/testReport/
Max. process+thread count 2105 (vs. ulimit of 5500)
modules C: tez-api tez-common tez-runtime-internals tez-runtime-library tez-mapreduce tez-dag tez-ext-service-tests tez-plugins/tez-history-parser tez-tools/analyzers/job-analyzer . U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-459/1/console
versions git=2.43.0 maven=3.8.7 spotbugs=4.9.3 codespell=2.4.1
Powered by Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

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.

3 participants