Skip to content

HBASE-28927 Fix spotbugs issues introduced by refactoring to hbase-diagnostics with HBASE-28432 - #6387

Merged
NihalJain merged 1 commit into
apache:branch-3from
NihalJain:HBASE-28927_branch-3
Oct 24, 2024
Merged

HBASE-28927 Fix spotbugs issues introduced by refactoring to hbase-diagnostics with HBASE-28432#6387
NihalJain merged 1 commit into
apache:branch-3from
NihalJain:HBASE-28927_branch-3

Conversation

@NihalJain

Copy link
Copy Markdown
Contributor

…agnostics with HBASE-28432 (apache#6384)
Signed-off-by: Istvan Toth <stoty@apache.org>
(cherry picked from commit f4aafaf)
@NihalJainNihalJain added the backport This PR is a back port of some issue or issues already committed to master label Oct 23, 2024
@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 40sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ branch-3 Compile Tests _
+0 🆗mvndep0m 8sMaven dependency ordering for branch
+1 💚mvninstall3m 0sbranch-3 passed
+1 💚compile3m 23sbranch-3 passed
+1 💚checkstyle0m 50sbranch-3 passed
-1 ❌spotbugs0m 32s/branch-spotbugs-hbase-diagnostics-warnings.htmlhbase-diagnostics in branch-3 has 12 extant spotbugs warnings.
-1 ❌spotbugs1m 30s/branch-spotbugs-hbase-server-warnings.htmlhbase-server in branch-3 has 1 extant spotbugs warnings.
+1 💚spotless0m 44sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 52sthe patch passed
+1 💚compile3m 21sthe patch passed
-0 ⚠️javac0m 27s/results-compile-javac-hbase-diagnostics.txthbase-diagnostics generated 1 new + 55 unchanged - 5 fixed = 56 total (was 60)
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 36sThe patch passed checkstyle in hbase-server
+1 💚checkstyle0m 10shbase-diagnostics: The patch generated 0 new + 43 unchanged - 1 fixed = 43 total (was 44)
+1 💚spotbugs1m 39shbase-server generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚spotbugs0m 36shbase-diagnostics generated 0 new + 0 unchanged - 12 fixed = 0 total (was 12)
+1 💚hadoopcheck10m 19sPatch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚spotless0m 43spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 20sThe patch does not generate ASF License warnings.
38m 17s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6387/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6387
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux eebf4f742bc2 5.4.0-192-generic #212-Ubuntu SMP Fri Jul 5 09:47:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-3 / 22c2047
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count84 (vs. ulimit of 30000)
modulesC: hbase-server hbase-diagnostics U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6387/1/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 29sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-3 Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall3m 16sbranch-3 passed
+1 💚compile1m 16sbranch-3 passed
+1 💚javadoc0m 42sbranch-3 passed
+1 💚shadedjars5m 48sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall3m 4sthe patch passed
+1 💚compile1m 20sthe patch passed
+1 💚javac1m 20sthe patch passed
+1 💚javadoc0m 45sthe patch passed
+1 💚shadedjars7m 45spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit225m 45shbase-server in the patch passed.
+1 💚unit5m 55shbase-diagnostics in the patch passed.
261m 30s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6387/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6387
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 6b27fe526d27 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-3 / 22c2047
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6387/1/testReport/
Max. process+thread count5057 (vs. ulimit of 30000)
modulesC: hbase-server hbase-diagnostics U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6387/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@NihalJain
NihalJain merged commit acb6fae into apache:branch-3Oct 24, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backportThis PR is a back port of some issue or issues already committed to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@NihalJain@Apache-HBase