Skip to content

TEZ-4734: Fix Checkstyle reported in Junit6 and few extra deprecated methods#517

Open
Aggarwal-Raghav wants to merge 16 commits into
apache:masterfrom
Aggarwal-Raghav:TEZ-4734
Open

TEZ-4734: Fix Checkstyle reported in Junit6 and few extra deprecated methods#517
Aggarwal-Raghav wants to merge 16 commits into
apache:masterfrom
Aggarwal-Raghav:TEZ-4734

Conversation

@Aggarwal-Raghav

Copy link
Copy Markdown
Contributor

No description provided.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 2m 7s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s 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 62 new or modified test files.
_ master Compile Tests _
+0 🆗 mvndep 0m 31s Maven dependency ordering for branch
+1 💚 mvninstall 2m 52s master passed
+1 💚 compile 2m 33s master passed
+1 💚 checkstyle 4m 24s master passed
+1 💚 javadoc 4m 28s master passed
+0 🆗 spotbugs 0m 50s tez-api in master has 92 extant spotbugs warnings.
+0 🆗 spotbugs 0m 34s tez-runtime-internals in master has 92 extant spotbugs warnings.
+0 🆗 spotbugs 0m 43s tez-runtime-library in master has 143 extant spotbugs warnings.
+0 🆗 spotbugs 0m 35s tez-mapreduce in master has 97 extant spotbugs warnings.
+0 🆗 spotbugs 0m 57s tez-dag in master has 537 extant spotbugs warnings.
+0 🆗 spotbugs 0m 31s tez-tests in master has 6 extant spotbugs warnings.
+0 🆗 spotbugs 0m 32s tez-plugins/tez-protobuf-history-plugin in master has 7 extant spotbugs warnings.
+0 🆗 spotbugs 0m 29s tez-plugins/tez-yarn-timeline-history in master has 2 extant spotbugs warnings.
+0 🆗 spotbugs 0m 28s tez-plugins/tez-yarn-timeline-history-with-acls in master has 2 extant spotbugs warnings.
+0 🆗 spotbugs 0m 27s tez-plugins/tez-yarn-timeline-cache-plugin in master has 1 extant spotbugs warnings.
+0 🆗 spotbugs 0m 29s tez-plugins/tez-yarn-timeline-history-with-fs in master has 5 extant spotbugs warnings.
+0 🆗 spotbugs 0m 31s tez-plugins/tez-history-parser in master has 21 extant spotbugs warnings.
+0 🆗 spotbugs 0m 30s tez-plugins/tez-aux-services in master has 5 extant spotbugs warnings.
+0 🆗 spotbugs 0m 33s tez-tools/analyzers/job-analyzer in master has 20 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 7s Maven dependency ordering for patch
+1 💚 mvninstall 2m 22s the patch passed
+1 💚 codespell 0m 53s No new issues.
+1 💚 compile 2m 31s the patch passed
+1 💚 javac 2m 31s root generated 0 new + 681 unchanged - 25 fixed = 681 total (was 706)
+1 💚 blanks 0m 0s The patch has no blanks issues.
-0 ⚠️ checkstyle 0m 21s /results-checkstyle-tez-api.txt tez-api: The patch generated 3 new + 75 unchanged - 34 fixed = 78 total (was 109)
-0 ⚠️ checkstyle 0m 18s /results-checkstyle-tez-runtime-internals.txt tez-runtime-internals: The patch generated 1 new + 32 unchanged - 26 fixed = 33 total (was 58)
-0 ⚠️ checkstyle 0m 21s /results-checkstyle-tez-runtime-library.txt tez-runtime-library: The patch generated 7 new + 336 unchanged - 265 fixed = 343 total (was 601)
-0 ⚠️ checkstyle 0m 21s /results-checkstyle-tez-mapreduce.txt tez-mapreduce: The patch generated 3 new + 877 unchanged - 35 fixed = 880 total (was 912)
-0 ⚠️ checkstyle 0m 24s /results-checkstyle-tez-dag.txt tez-dag: The patch generated 9 new + 572 unchanged - 98 fixed = 581 total (was 670)
+1 💚 checkstyle 0m 19s tez-tests: The patch generated 0 new + 99 unchanged - 35 fixed = 99 total (was 134)
+1 💚 checkstyle 0m 16s tez-plugins/tez-protobuf-history-plugin: The patch generated 0 new + 14 unchanged - 11 fixed = 14 total (was 25)
-0 ⚠️ checkstyle 0m 17s /results-checkstyle-tez-plugins_tez-yarn-timeline-history.txt tez-plugins/tez-yarn-timeline-history: The patch generated 4 new + 26 unchanged - 20 fixed = 30 total (was 46)
+1 💚 checkstyle 0m 16s tez-plugins/tez-yarn-timeline-history-with-acls: The patch generated 0 new + 10 unchanged - 8 fixed = 10 total (was 18)
+1 💚 checkstyle 0m 14s tez-plugins/tez-yarn-timeline-cache-plugin: The patch generated 0 new + 16 unchanged - 2 fixed = 16 total (was 18)
+1 💚 checkstyle 0m 16s tez-plugins/tez-yarn-timeline-history-with-fs: The patch generated 0 new + 24 unchanged - 4 fixed = 24 total (was 28)
+1 💚 checkstyle 0m 17s tez-plugins/tez-history-parser: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 💚 checkstyle 0m 18s tez-plugins/tez-aux-services: The patch generated 0 new + 31 unchanged - 1 fixed = 31 total (was 32)
-0 ⚠️ checkstyle 0m 16s /results-checkstyle-tez-tools_analyzers_job-analyzer.txt tez-tools/analyzers/job-analyzer: The patch generated 1 new + 9 unchanged - 3 fixed = 10 total (was 12)
+1 💚 javadoc 4m 29s the patch passed
+1 💚 spotbugs 10m 10s the patch passed
_ Other Tests _
+1 💚 unit 62m 48s root in the patch passed.
+1 💚 asflicense 4m 22s The patch does not generate ASF License warnings.
121m 26s
Subsystem Report/Notes
Docker ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-517/1/artifact/out/Dockerfile
Optional Tests dupname compile unit asflicense javac javadoc spotbugs checkstyle codespell detsecrets
uname Linux 0b42bf2c708f 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality tez-personality.sh
git revision master / d53e67d
Default Java Eclipse Adoptium-21.0.11+10-LTS
Test Results https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-517/1/testReport/
Max. process+thread count 1427 (vs. ulimit of 5500)
modules C: tez-api tez-runtime-internals tez-runtime-library tez-mapreduce tez-dag tez-tests tez-plugins/tez-protobuf-history-plugin tez-plugins/tez-yarn-timeline-history tez-plugins/tez-yarn-timeline-history-with-acls tez-plugins/tez-yarn-timeline-cache-plugin tez-plugins/tez-yarn-timeline-history-with-fs tez-plugins/tez-history-parser tez-plugins/tez-aux-services tez-tools/analyzers/job-analyzer U: .
Console output https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-517/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.

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