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

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec2m 7sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo 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 62 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep0m 31sMaven dependency ordering for branch
+1 💚mvninstall2m 52smaster passed
+1 💚compile2m 33smaster passed
+1 💚checkstyle4m 24smaster passed
+1 💚javadoc4m 28smaster passed
+0 🆗spotbugs0m 50stez-api in master has 92 extant spotbugs warnings.
+0 🆗spotbugs0m 34stez-runtime-internals in master has 92 extant spotbugs warnings.
+0 🆗spotbugs0m 43stez-runtime-library in master has 143 extant spotbugs warnings.
+0 🆗spotbugs0m 35stez-mapreduce in master has 97 extant spotbugs warnings.
+0 🆗spotbugs0m 57stez-dag in master has 537 extant spotbugs warnings.
+0 🆗spotbugs0m 31stez-tests in master has 6 extant spotbugs warnings.
+0 🆗spotbugs0m 32stez-plugins/tez-protobuf-history-plugin in master has 7 extant spotbugs warnings.
+0 🆗spotbugs0m 29stez-plugins/tez-yarn-timeline-history in master has 2 extant spotbugs warnings.
+0 🆗spotbugs0m 28stez-plugins/tez-yarn-timeline-history-with-acls in master has 2 extant spotbugs warnings.
+0 🆗spotbugs0m 27stez-plugins/tez-yarn-timeline-cache-plugin in master has 1 extant spotbugs warnings.
+0 🆗spotbugs0m 29stez-plugins/tez-yarn-timeline-history-with-fs in master has 5 extant spotbugs warnings.
+0 🆗spotbugs0m 31stez-plugins/tez-history-parser in master has 21 extant spotbugs warnings.
+0 🆗spotbugs0m 30stez-plugins/tez-aux-services in master has 5 extant spotbugs warnings.
+0 🆗spotbugs0m 33stez-tools/analyzers/job-analyzer in master has 20 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 7sMaven dependency ordering for patch
+1 💚mvninstall2m 22sthe patch passed
+1 💚codespell0m 53sNo new issues.
+1 💚compile2m 31sthe patch passed
+1 💚javac2m 31sroot generated 0 new + 681 unchanged - 25 fixed = 681 total (was 706)
+1 💚blanks0m 0sThe patch has no blanks issues.
-0 ⚠️checkstyle0m 21s/results-checkstyle-tez-api.txttez-api: The patch generated 3 new + 75 unchanged - 34 fixed = 78 total (was 109)
-0 ⚠️checkstyle0m 18s/results-checkstyle-tez-runtime-internals.txttez-runtime-internals: The patch generated 1 new + 32 unchanged - 26 fixed = 33 total (was 58)
-0 ⚠️checkstyle0m 21s/results-checkstyle-tez-runtime-library.txttez-runtime-library: The patch generated 7 new + 336 unchanged - 265 fixed = 343 total (was 601)
-0 ⚠️checkstyle0m 21s/results-checkstyle-tez-mapreduce.txttez-mapreduce: The patch generated 3 new + 877 unchanged - 35 fixed = 880 total (was 912)
-0 ⚠️checkstyle0m 24s/results-checkstyle-tez-dag.txttez-dag: The patch generated 9 new + 572 unchanged - 98 fixed = 581 total (was 670)
+1 💚checkstyle0m 19stez-tests: The patch generated 0 new + 99 unchanged - 35 fixed = 99 total (was 134)
+1 💚checkstyle0m 16stez-plugins/tez-protobuf-history-plugin: The patch generated 0 new + 14 unchanged - 11 fixed = 14 total (was 25)
-0 ⚠️checkstyle0m 17s/results-checkstyle-tez-plugins_tez-yarn-timeline-history.txttez-plugins/tez-yarn-timeline-history: The patch generated 4 new + 26 unchanged - 20 fixed = 30 total (was 46)
+1 💚checkstyle0m 16stez-plugins/tez-yarn-timeline-history-with-acls: The patch generated 0 new + 10 unchanged - 8 fixed = 10 total (was 18)
+1 💚checkstyle0m 14stez-plugins/tez-yarn-timeline-cache-plugin: The patch generated 0 new + 16 unchanged - 2 fixed = 16 total (was 18)
+1 💚checkstyle0m 16stez-plugins/tez-yarn-timeline-history-with-fs: The patch generated 0 new + 24 unchanged - 4 fixed = 24 total (was 28)
+1 💚checkstyle0m 17stez-plugins/tez-history-parser: The patch generated 0 new + 12 unchanged - 1 fixed = 12 total (was 13)
+1 💚checkstyle0m 18stez-plugins/tez-aux-services: The patch generated 0 new + 31 unchanged - 1 fixed = 31 total (was 32)
-0 ⚠️checkstyle0m 16s/results-checkstyle-tez-tools_analyzers_job-analyzer.txttez-tools/analyzers/job-analyzer: The patch generated 1 new + 9 unchanged - 3 fixed = 10 total (was 12)
+1 💚javadoc4m 29sthe patch passed
+1 💚spotbugs10m 10sthe patch passed
_ Other Tests _
+1 💚unit62m 48sroot in the patch passed.
+1 💚asflicense4m 22sThe patch does not generate ASF License warnings.
121m 26s
SubsystemReport/Notes
DockerClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-517/1/artifact/out/Dockerfile
Optional Testsdupname compile unit asflicense javac javadoc spotbugs checkstyle codespell detsecrets
unameLinux 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 toolmaven
Personalitytez-personality.sh
git revisionmaster / d53e67d
Default JavaEclipse Adoptium-21.0.11+10-LTS
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-517/1/testReport/
Max. process+thread count1427 (vs. ulimit of 5500)
modulesC: 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 outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-517/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.

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.

2 participants

@Aggarwal-Raghav@tez-yetus