Skip to content

Conversation

sairampola
Copy link

No description provided.

sairampola added 30 commits April 15, 2025 19:31
@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

1 similar comment
@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 31s Docker mode activated.
-0 ⚠️ yetus 0m 3s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ HBASE-29588-feature Compile Tests _
+0 🆗 mvndep 0m 22s Maven dependency ordering for branch
+1 💚 mvninstall 3m 14s HBASE-29588-feature passed
+1 💚 compile 1m 56s HBASE-29588-feature passed
+1 💚 javadoc 0m 53s HBASE-29588-feature passed
+1 💚 shadedjars 6m 5s branch has no errors when building our shaded downstream artifacts.
-0 ⚠️ patch 6m 29s Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 13s Maven dependency ordering for patch
+1 💚 mvninstall 3m 7s the patch passed
+1 💚 compile 1m 51s the patch passed
+1 💚 javac 1m 51s the patch passed
-0 ⚠️ javadoc 0m 27s /results-javadoc-javadoc-hbase-server.txt hbase-server generated 1 new + 63 unchanged - 0 fixed = 64 total (was 63)
+1 💚 shadedjars 6m 4s patch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚 unit 0m 34s hbase-protocol-shaded in the patch passed.
+1 💚 unit 2m 20s hbase-common in the patch passed.
-1 ❌ unit 9m 21s /patch-unit-hbase-server.txt hbase-server in the patch failed.
39m 28s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7296/14/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR #7296
JIRA Issue HBASE-29588
Optional Tests javac javadoc unit compile shadedjars
uname Linux 2061f53ee4a6 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 tool maven
Personality dev-support/hbase-personality.sh
git revision HBASE-29588-feature / d0ef67d
Default Java Eclipse Adoptium-17.0.11+9
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7296/14/testReport/
Max. process+thread count 1198 (vs. ulimit of 30000)
modules C: hbase-protocol-shaded hbase-common hbase-server U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7296/14/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 31s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 buf 0m 0s buf was not available.
+0 🆗 buf 0m 0s buf was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
_ HBASE-29588-feature Compile Tests _
+0 🆗 mvndep 0m 20s Maven dependency ordering for branch
+1 💚 mvninstall 3m 11s HBASE-29588-feature passed
+1 💚 compile 4m 41s HBASE-29588-feature passed
+1 💚 checkstyle 0m 57s HBASE-29588-feature passed
+1 💚 spotbugs 4m 22s HBASE-29588-feature passed
+1 💚 spotless 0m 45s branch has no errors when running spotless:check.
-0 ⚠️ patch 1m 3s Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 12s Maven dependency ordering for patch
+1 💚 mvninstall 3m 9s the patch passed
-1 ❌ compile 1m 32s /patch-compile-hbase-server.txt hbase-server in the patch failed.
-1 ❌ cc 1m 32s /patch-compile-hbase-server.txt hbase-server in the patch failed.
-0 ⚠️ javac 1m 32s /patch-compile-hbase-server.txt hbase-server in the patch failed.
+1 💚 blanks 0m 1s The patch has no blanks issues.
-0 ⚠️ checkstyle 0m 37s /results-checkstyle-hbase-server.txt hbase-server: The patch generated 20 new + 19 unchanged - 0 fixed = 39 total (was 19)
+1 💚 xmllint 0m 0s No new issues.
-1 ❌ spotbugs 2m 11s /new-spotbugs-hbase-server.html hbase-server generated 10 new + 0 unchanged - 0 fixed = 10 total (was 0)
+1 💚 hadoopcheck 12m 14s Patch does not cause any errors with Hadoop 3.3.6 3.4.1.
+1 💚 hbaseprotoc 1m 35s the patch passed
+1 💚 spotless 0m 45s patch has no errors when running spotless:check.
_ Other Tests _
+1 💚 asflicense 0m 26s The patch does not generate ASF License warnings.
50m 11s
Reason Tests
SpotBugs module:hbase-server
Load of known null value in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getFirstKey() At AbstractMultiTenantReader.java:in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getFirstKey() At AbstractMultiTenantReader.java:[line 1256]
Load of known null value in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getLastKey() At AbstractMultiTenantReader.java:in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getLastKey() At AbstractMultiTenantReader.java:[line 1293]
Load of known null value in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadDataBlockIndexFromSection(byte[]) At AbstractMultiTenantReader.java:in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadDataBlockIndexFromSection(byte[]) At AbstractMultiTenantReader.java:[line 1399]
Load of known null value in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadMetaBlockFromSection(byte[], String, boolean) At AbstractMultiTenantReader.java:in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadMetaBlockFromSection(byte[], String, boolean) At AbstractMultiTenantReader.java:[line 1418]
Redundant nullcheck of lease which is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getFirstKey() Redundant null check at AbstractMultiTenantReader.java:is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getFirstKey() Redundant null check at AbstractMultiTenantReader.java:[line 1256]
Redundant nullcheck of lease which is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getLastKey() Redundant null check at AbstractMultiTenantReader.java:is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.getLastKey() Redundant null check at AbstractMultiTenantReader.java:[line 1293]
Redundant nullcheck of lease which is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadDataBlockIndexFromSection(byte[]) Redundant null check at AbstractMultiTenantReader.java:is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadDataBlockIndexFromSection(byte[]) Redundant null check at AbstractMultiTenantReader.java:[line 1399]
Redundant nullcheck of lease which is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadMetaBlockFromSection(byte[], String, boolean) Redundant null check at AbstractMultiTenantReader.java:is known to be null in org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader.loadMetaBlockFromSection(byte[], String, boolean) Redundant null check at AbstractMultiTenantReader.java:[line 1418]
Should org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader$SectionReaderHolder be a static inner class? At AbstractMultiTenantReader.java:inner class? At AbstractMultiTenantReader.java:[lines 624-684]
Should org.apache.hadoop.hbase.io.hfile.AbstractMultiTenantReader$SectionReaderLease be a static inner class? At AbstractMultiTenantReader.java:inner class? At AbstractMultiTenantReader.java:[lines 697-730]
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7296/14/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #7296
JIRA Issue HBASE-29588
Optional Tests dupname asflicense javac codespell detsecrets xmllint spotless cc buflint bufcompat hbaseprotoc spotbugs checkstyle compile hadoopcheck hbaseanti
uname Linux 1c897bce5f79 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 tool maven
Personality dev-support/hbase-personality.sh
git revision HBASE-29588-feature / d0ef67d
Default Java Eclipse Adoptium-17.0.11+9
Max. process+thread count 89 (vs. ulimit of 30000)
modules C: hbase-protocol-shaded hbase-common hbase-server U: .
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7296/14/console
versions git=2.34.1 maven=3.9.8 spotbugs=4.7.3 xmllint=20913
Powered by Apache Yetus 0.15.0 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