Skip to content

HBASE-28644 Use ExtendedCell instead of Cell in KeyValueScanner - #5976

Merged
Apache9 merged 1 commit into
apache:masterfrom
Apache9:HBASE-28644
Jun 21, 2024
Merged

HBASE-28644 Use ExtendedCell instead of Cell in KeyValueScanner#5976
Apache9 merged 1 commit into
apache:masterfrom
Apache9:HBASE-28644

Conversation

@Apache9

Copy link
Copy Markdown
Contributor

No description provided.

@Apache9Apache9 self-assigned this Jun 11, 2024
@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 39sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for branch
+1 💚mvninstall4m 40smaster passed
+1 💚compile5m 45smaster passed
+1 💚checkstyle1m 33smaster passed
+1 💚spotless0m 55sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 9smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall3m 56sthe patch passed
+1 💚compile5m 33sthe patch passed
+1 💚javac5m 33sthe patch passed
-0 ⚠️checkstyle0m 43shbase-server: The patch generated 2 new + 133 unchanged - 9 fixed = 135 total (was 142)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck7m 25sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless1m 18spatch has no errors when running spotless:check.
-1 ❌spotbugs2m 52shbase-server generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)
_ Other Tests _
+1 💚asflicense0m 51sThe patch does not generate ASF License warnings.
57m 27s
ReasonTests
FindBugsmodule:hbase-server
instanceof will always return true for all non-null values in org.apache.hadoop.hbase.regionserver.MemStoreLABImpl.copyToChunkCell(ExtendedCell, ByteBuffer, int, int), since all org.apache.hadoop.hbase.ExtendedCell are instances of org.apache.hadoop.hbase.ExtendedCell At MemStoreLABImpl.java:for all non-null values in org.apache.hadoop.hbase.regionserver.MemStoreLABImpl.copyToChunkCell(ExtendedCell, ByteBuffer, int, int), since all org.apache.hadoop.hbase.ExtendedCell are instances of org.apache.hadoop.hbase.ExtendedCell At MemStoreLABImpl.java:[line 222]
Unchecked/unconfirmed cast from org.apache.hadoop.hbase.coprocessor.ObserverContext<org.apache.hadoop.hbase.coprocessor.RegionCoprocessorEnvironment> to org.apache.hadoop.hbase.ExtendedCell in org.apache.hadoop.hbase.security.visibility.VisibilityController.preBatchMutate(ObserverContext, MiniBatchOperationInProgress) At VisibilityController.java:org.apache.hadoop.hbase.ExtendedCell in org.apache.hadoop.hbase.security.visibility.VisibilityController.preBatchMutate(ObserverContext, MiniBatchOperationInProgress) At VisibilityController.java:[line 348]
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5976
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux c7c946318d09 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 revisionmaster / c5d3f65
Default JavaEclipse Adoptium-11.0.17+8
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
spotbugshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-general-check/output/new-spotbugs-hbase-server.html
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 29sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for branch
+1 💚mvninstall2m 36smaster passed
+1 💚compile1m 25smaster passed
+1 💚shadedjars5m 34sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 2smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 23sthe patch passed
+1 💚compile1m 29sthe patch passed
+1 💚javac1m 29sthe patch passed
+1 💚shadedjars5m 34spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 22shbase-server generated 16 new + 22 unchanged - 0 fixed = 38 total (was 22)
_ Other Tests _
+1 💚unit1m 53shbase-common in the patch passed.
+1 💚unit1m 25shbase-client in the patch passed.
-1 ❌unit11m 2shbase-server in the patch failed.
-1 ❌unit22m 22shbase-mapreduce in the patch failed.
61m 34s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux faeb6cd1ef35 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 revisionmaster / c5d3f65
Default JavaTemurin-1.8.0_352-b08
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-mapreduce.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/testReport/
Max. process+thread count2874 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 28sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall3m 10smaster passed
+1 💚compile1m 51smaster passed
+1 💚shadedjars5m 43sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 8smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 57sthe patch passed
+1 💚compile1m 51sthe patch passed
+1 💚javac1m 51sthe patch passed
+1 💚shadedjars5m 40spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 25shbase-server generated 41 new + 48 unchanged - 0 fixed = 89 total (was 48)
_ Other Tests _
+1 💚unit2m 9shbase-common in the patch passed.
+1 💚unit1m 33shbase-client in the patch passed.
-1 ❌unit12m 11shbase-server in the patch failed.
-1 ❌unit21m 55shbase-mapreduce in the patch failed.
64m 58s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux b62d70de78ad 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 revisionmaster / c5d3f65
Default JavaEclipse Adoptium-17.0.10+7
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk17-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-mapreduce.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/testReport/
Max. process+thread count2970 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 41sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for branch
+1 💚mvninstall4m 43smaster passed
+1 💚compile2m 25smaster passed
+1 💚shadedjars6m 58sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 39smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall4m 3sthe patch passed
+1 💚compile2m 22sthe patch passed
+1 💚javac2m 22sthe patch passed
+1 💚shadedjars6m 52spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 34shbase-server generated 21 new + 71 unchanged - 25 fixed = 92 total (was 96)
_ Other Tests _
+1 💚unit2m 49shbase-common in the patch passed.
+1 💚unit1m 51shbase-client in the patch passed.
-1 ❌unit18m 16shbase-server in the patch failed.
-1 ❌unit22m 59shbase-mapreduce in the patch failed.
80m 52s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 83e50e0e5a29 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 revisionmaster / c5d3f65
Default JavaEclipse Adoptium-11.0.17+8
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk11-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-mapreduce.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/testReport/
Max. process+thread count2943 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/1/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 42sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for branch
+1 💚mvninstall4m 44smaster passed
+1 💚compile5m 50smaster passed
+1 💚checkstyle1m 32smaster passed
+1 💚spotless0m 55sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 16smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall4m 5sthe patch passed
+1 💚compile5m 41sthe patch passed
+1 💚javac5m 41sthe patch passed
-0 ⚠️checkstyle0m 42shbase-server: The patch generated 1 new + 133 unchanged - 9 fixed = 134 total (was 142)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck7m 51sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless1m 17spatch has no errors when running spotless:check.
+1 💚spotbugs6m 50sthe patch passed
_ Other Tests _
+1 💚asflicense0m 49sThe patch does not generate ASF License warnings.
58m 29s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5976
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux 564a87a41312 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 revisionmaster / c5d3f65
Default JavaEclipse Adoptium-11.0.17+8
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count81 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 41sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for branch
+1 💚mvninstall4m 39smaster passed
+1 💚compile2m 23smaster passed
+1 💚shadedjars7m 2sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 37smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall4m 3sthe patch passed
+1 💚compile2m 23sthe patch passed
+1 💚javac2m 23sthe patch passed
+1 💚shadedjars7m 5spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 34shbase-server generated 1 new + 96 unchanged - 0 fixed = 97 total (was 96)
_ Other Tests _
+1 💚unit2m 51shbase-common in the patch passed.
+1 💚unit1m 52shbase-client in the patch passed.
-1 ❌unit433m 16shbase-server in the patch failed.
+1 💚unit15m 12shbase-mapreduce in the patch passed.
491m 19s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 2dcb8eab5ee6 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 revisionmaster / c5d3f65
Default JavaEclipse Adoptium-11.0.17+8
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk11-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/testReport/
Max. process+thread count6153 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 32sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for branch
+1 💚mvninstall3m 40smaster passed
+1 💚compile2m 15smaster passed
+1 💚shadedjars6m 33sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 23smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall3m 13sthe patch passed
+1 💚compile2m 11sthe patch passed
+1 💚javac2m 11sthe patch passed
+1 💚shadedjars6m 30spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 29shbase-server generated 1 new + 48 unchanged - 0 fixed = 49 total (was 48)
_ Other Tests _
+1 💚unit2m 46shbase-common in the patch passed.
+1 💚unit1m 55shbase-client in the patch passed.
-1 ❌unit439m 49shbase-server in the patch failed.
+1 💚unit16m 17shbase-mapreduce in the patch passed.
494m 53s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 6d9a89608498 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 revisionmaster / c5d3f65
Default JavaEclipse Adoptium-17.0.10+7
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk17-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/testReport/
Max. process+thread count6108 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 27sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall2m 21smaster passed
+1 💚compile1m 27smaster passed
+1 💚shadedjars5m 35sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 59smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 22sthe patch passed
+1 💚compile1m 27sthe patch passed
+1 💚javac1m 27sthe patch passed
+1 💚shadedjars5m 33spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 22shbase-server generated 2 new + 22 unchanged - 0 fixed = 24 total (was 22)
_ Other Tests _
+1 💚unit1m 52shbase-common in the patch passed.
+1 💚unit1m 24shbase-client in the patch passed.
-1 ❌unit464m 46shbase-server in the patch failed.
+1 💚unit16m 9shbase-mapreduce in the patch passed.
511m 49s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux da237a0ad0b2 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 revisionmaster / c5d3f65
Default JavaTemurin-1.8.0_352-b08
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/testReport/
Max. process+thread count5902 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/2/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 44sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 35sMaven dependency ordering for branch
+1 💚mvninstall4m 50smaster passed
+1 💚compile5m 37smaster passed
+1 💚checkstyle1m 29smaster passed
+1 💚spotless0m 57sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 15smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall3m 55sthe patch passed
+1 💚compile5m 24sthe patch passed
+1 💚javac5m 24sthe patch passed
-0 ⚠️checkstyle0m 42shbase-server: The patch generated 1 new + 133 unchanged - 9 fixed = 134 total (was 142)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck7m 41sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless1m 19spatch has no errors when running spotless:check.
+1 💚spotbugs6m 36sthe patch passed
_ Other Tests _
+1 💚asflicense0m 46sThe patch does not generate ASF License warnings.
57m 48s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5976
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux fcac9de20d06 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 revisionmaster / 317ad3c
Default JavaEclipse Adoptium-11.0.17+8
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count77 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 28sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for branch
+1 💚mvninstall3m 9smaster passed
+1 💚compile1m 51smaster passed
+1 💚shadedjars5m 45sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 9smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 57sthe patch passed
+1 💚compile1m 51sthe patch passed
+1 💚javac1m 51sthe patch passed
+1 💚shadedjars5m 42spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 25shbase-server generated 1 new + 48 unchanged - 0 fixed = 49 total (was 48)
_ Other Tests _
+1 💚unit2m 12shbase-common in the patch passed.
+1 💚unit1m 34shbase-client in the patch passed.
+1 💚unit218m 44shbase-server in the patch passed.
+1 💚unit14m 15shbase-mapreduce in the patch passed.
266m 39s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux e96ecabf9e50 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 revisionmaster / 317ad3c
Default JavaEclipse Adoptium-17.0.10+7
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-jdk17-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/testReport/
Max. process+thread count4978 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 27sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 22sMaven dependency ordering for branch
+1 💚mvninstall2m 27smaster passed
+1 💚compile1m 27smaster passed
+1 💚shadedjars5m 40sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 0smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 29sthe patch passed
+1 💚compile1m 27sthe patch passed
+1 💚javac1m 27sthe patch passed
+1 💚shadedjars5m 39spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 22shbase-server generated 2 new + 22 unchanged - 0 fixed = 24 total (was 22)
_ Other Tests _
+1 💚unit1m 54shbase-common in the patch passed.
+1 💚unit1m 25shbase-client in the patch passed.
+1 💚unit234m 28shbase-server in the patch passed.
+1 💚unit14m 54shbase-mapreduce in the patch passed.
280m 22s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux e71160b893b1 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 revisionmaster / 317ad3c
Default JavaTemurin-1.8.0_352-b08
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/testReport/
Max. process+thread count5230 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 42sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 35sMaven dependency ordering for branch
+1 💚mvninstall4m 41smaster passed
+1 💚compile2m 21smaster passed
+1 💚shadedjars6m 48sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 36smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall3m 55sthe patch passed
+1 💚compile2m 19sthe patch passed
+1 💚javac2m 19sthe patch passed
+1 💚shadedjars6m 48spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 33shbase-server generated 1 new + 96 unchanged - 0 fixed = 97 total (was 96)
_ Other Tests _
+1 💚unit2m 55shbase-common in the patch passed.
+1 💚unit1m 50shbase-client in the patch passed.
+1 💚unit236m 39shbase-server in the patch passed.
+1 💚unit15m 31shbase-mapreduce in the patch passed.
294m 29s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 306115aef229 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 revisionmaster / 317ad3c
Default JavaEclipse Adoptium-11.0.17+8
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/artifact/yetus-jdk11-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/testReport/
Max. process+thread count4434 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/3/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 43sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 40sMaven dependency ordering for branch
+1 💚mvninstall4m 49smaster passed
+1 💚compile5m 45smaster passed
+1 💚checkstyle1m 37smaster passed
+1 💚spotless0m 54sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 37smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall4m 1sthe patch passed
+1 💚compile5m 23sthe patch passed
+1 💚javac5m 23sthe patch passed
+1 💚checkstyle0m 16sThe patch passed checkstyle in hbase-common
+1 💚checkstyle0m 18sThe patch passed checkstyle in hbase-client
+1 💚checkstyle0m 38shbase-server: The patch generated 0 new + 133 unchanged - 9 fixed = 133 total (was 142)
+1 💚checkstyle0m 14sThe patch passed checkstyle in hbase-mapreduce
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck7m 54sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless1m 20spatch has no errors when running spotless:check.
+1 💚spotbugs6m 47sthe patch passed
_ Other Tests _
+1 💚asflicense0m 46sThe patch does not generate ASF License warnings.
59m 13s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5976
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux b84b265949c7 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 revisionmaster / 9bdee6d
Default JavaEclipse Adoptium-11.0.17+8
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 55sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall3m 36smaster passed
+1 💚compile2m 24smaster passed
+1 💚shadedjars7m 48sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 24smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for patch
+1 💚mvninstall3m 52sthe patch passed
+1 💚compile2m 22sthe patch passed
+1 💚javac2m 22sthe patch passed
+1 💚shadedjars7m 34spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 45sthe patch passed
_ Other Tests _
+1 💚unit2m 58shbase-common in the patch passed.
+1 💚unit2m 7shbase-client in the patch passed.
+1 💚unit235m 18shbase-server in the patch passed.
+1 💚unit18m 17shbase-mapreduce in the patch passed.
296m 53s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 8c77ab735ed1 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 revisionmaster / 9bdee6d
Default JavaTemurin-1.8.0_352-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/testReport/
Max. process+thread count5333 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 36sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for branch
+1 💚mvninstall3m 23smaster passed
+1 💚compile2m 19smaster passed
+1 💚shadedjars6m 21sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 26smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 9sMaven dependency ordering for patch
+1 💚mvninstall3m 14sthe patch passed
+1 💚compile2m 8sthe patch passed
+1 💚javac2m 8sthe patch passed
+1 💚shadedjars6m 28spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 27sthe patch passed
_ Other Tests _
+1 💚unit2m 41shbase-common in the patch passed.
+1 💚unit1m 56shbase-client in the patch passed.
+1 💚unit244m 33shbase-server in the patch passed.
+1 💚unit20m 42shbase-mapreduce in the patch passed.
303m 40s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 81902fbea21e 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 revisionmaster / 9bdee6d
Default JavaEclipse Adoptium-17.0.10+7
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/testReport/
Max. process+thread count5092 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 41sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 32sMaven dependency ordering for branch
+1 💚mvninstall4m 42smaster passed
+1 💚compile2m 19smaster passed
+1 💚shadedjars7m 5sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 41smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall4m 2sthe patch passed
+1 💚compile2m 26sthe patch passed
+1 💚javac2m 26sthe patch passed
+1 💚shadedjars6m 55spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 32sthe patch passed
_ Other Tests _
+1 💚unit2m 53shbase-common in the patch passed.
+1 💚unit1m 52shbase-client in the patch passed.
-1 ❌unit250m 40shbase-server in the patch failed.
+1 💚unit21m 15shbase-mapreduce in the patch passed.
315m 26s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux d274e27509d7 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 revisionmaster / 9bdee6d
Default JavaEclipse Adoptium-11.0.17+8
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/testReport/
Max. process+thread count5416 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/4/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@apurtellapurtell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes lgtm

I am not a fan of the type name ExtendedCell though. We are extending the cell interface into a new interface with extra methods that the server needs, so since we have to make breaking changes here why not call it ServerCell to more closely represent the real requirement. The comments in filter javadoc make a bit more sense then too because the returned cells are evaluated on the server side so need to be a ServerCell. That said, maybe someone else dislikes ServerCell as much as I dislike ExtendedCell and we don't need to get hung up on it. However given the breaking changes needed already this is our chance to make such changes.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

ExtendedCell is introduced in HBASE-16134.

We discussed about the naming, someone suggested ServersideCell but since the class have to be placed in hbase-common so ServersideCell was declined...

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 39sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 2sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 30sMaven dependency ordering for branch
+1 💚mvninstall4m 29smaster passed
+1 💚compile5m 48smaster passed
+1 💚checkstyle1m 33smaster passed
+1 💚spotless0m 53sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 27smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall3m 52sthe patch passed
+1 💚compile5m 28sthe patch passed
+1 💚javac5m 28sthe patch passed
+1 💚checkstyle0m 18sThe patch passed checkstyle in hbase-common
+1 💚checkstyle0m 20sThe patch passed checkstyle in hbase-client
+1 💚checkstyle0m 42shbase-server: The patch generated 0 new + 133 unchanged - 9 fixed = 133 total (was 142)
+1 💚checkstyle0m 16sThe patch passed checkstyle in hbase-mapreduce
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck8m 4sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless1m 20spatch has no errors when running spotless:check.
+1 💚spotbugs6m 41sthe patch passed
_ Other Tests _
+1 💚asflicense0m 48sThe patch does not generate ASF License warnings.
58m 34s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5976
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux 332b1af54dba 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 revisionmaster / 0e8cfdb
Default JavaEclipse Adoptium-11.0.17+8
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 27sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 21sMaven dependency ordering for branch
+1 💚mvninstall3m 5smaster passed
+1 💚compile1m 49smaster passed
+1 💚shadedjars5m 47sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 8smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 57sthe patch passed
+1 💚compile1m 50sthe patch passed
+1 💚javac1m 50sthe patch passed
+1 💚shadedjars5m 42spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 7sthe patch passed
_ Other Tests _
+1 💚unit2m 10shbase-common in the patch passed.
+1 💚unit1m 33shbase-client in the patch passed.
+1 💚unit235m 5shbase-server in the patch passed.
+1 💚unit19m 47shbase-mapreduce in the patch passed.
288m 38s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 449c6999e273 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 revisionmaster / 0e8cfdb
Default JavaEclipse Adoptium-17.0.10+7
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/testReport/
Max. process+thread count5671 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 29sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall2m 40smaster passed
+1 💚compile1m 27smaster passed
+1 💚shadedjars5m 39sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 3smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 25sthe patch passed
+1 💚compile1m 27sthe patch passed
+1 💚javac1m 27sthe patch passed
+1 💚shadedjars5m 38spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 0sthe patch passed
_ Other Tests _
+1 💚unit1m 54shbase-common in the patch passed.
+1 💚unit1m 25shbase-client in the patch passed.
-1 ❌unit263m 7shbase-server in the patch failed.
+1 💚unit18m 31shbase-mapreduce in the patch passed.
313m 9s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux f49322c8ab4d 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 revisionmaster / 0e8cfdb
Default JavaTemurin-1.8.0_352-b08
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/testReport/
Max. process+thread count4610 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 40sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 20sMaven dependency ordering for branch
+1 💚mvninstall4m 18smaster passed
+1 💚compile2m 17smaster passed
+1 💚shadedjars7m 3sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 35smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall3m 59sthe patch passed
+1 💚compile2m 27sthe patch passed
+1 💚javac2m 27sthe patch passed
+1 💚shadedjars7m 2spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 33sthe patch passed
_ Other Tests _
+1 💚unit2m 58shbase-common in the patch passed.
+1 💚unit1m 54shbase-client in the patch passed.
+1 💚unit262m 13shbase-server in the patch passed.
+1 💚unit22m 18shbase-mapreduce in the patch passed.
327m 31s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 93d295c267c9 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 revisionmaster / 0e8cfdb
Default JavaEclipse Adoptium-11.0.17+8
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/testReport/
Max. process+thread count4632 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/5/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 43sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall2m 59smaster passed
+1 💚compile4m 33smaster passed
+1 💚checkstyle1m 26smaster passed
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
+1 💚spotbugs3m 19smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 46sthe patch passed
+1 💚compile4m 32sthe patch passed
+1 💚javac4m 32sthe patch passed
+1 💚checkstyle0m 16sThe patch passed checkstyle in hbase-common
+1 💚checkstyle0m 16sThe patch passed checkstyle in hbase-client
+1 💚checkstyle0m 39shbase-server: The patch generated 0 new + 133 unchanged - 9 fixed = 133 total (was 142)
+1 💚checkstyle0m 12sThe patch passed checkstyle in hbase-mapreduce
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck5m 1sPatch does not cause any errors with Hadoop 3.3.6.
+1 💚spotless0m 42spatch has no errors when running spotless:check.
+1 💚spotbugs3m 53sthe patch passed
_ Other Tests _
+1 💚asflicense0m 39sThe patch does not generate ASF License warnings.
40m 27s
SubsystemReport/Notes
DockerClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5976
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux 9fb04bb164db 5.4.0-177-generic #197-Ubuntu SMP Thu Mar 28 22:45:47 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 52eef65
Default JavaEclipse Adoptium-11.0.17+8
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 27sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 38sMaven dependency ordering for branch
+1 💚mvninstall3m 18smaster passed
+1 💚compile1m 39smaster passed
+1 💚shadedjars5m 58sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 8smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall3m 2sthe patch passed
+1 💚compile1m 41sthe patch passed
+1 💚javac1m 41sthe patch passed
+1 💚shadedjars5m 50spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 6sthe patch passed
_ Other Tests _
+1 💚unit2m 20shbase-common in the patch passed.
+1 💚unit1m 38shbase-client in the patch passed.
-1 ❌unit225m 15shbase-server in the patch failed.
+1 💚unit18m 23shbase-mapreduce in the patch passed.
278m 22s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux bb80ed4c18f6 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 revisionmaster / 52eef65
Default JavaEclipse Adoptium-11.0.17+8
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/testReport/
Max. process+thread count4801 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 43sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 25sMaven dependency ordering for branch
+1 💚mvninstall4m 23smaster passed
+1 💚compile2m 31smaster passed
+1 💚shadedjars7m 14sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 48smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall3m 56sthe patch passed
+1 💚compile2m 22sthe patch passed
+1 💚javac2m 22sthe patch passed
+1 💚shadedjars7m 0spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 24sthe patch passed
_ Other Tests _
+1 💚unit2m 44shbase-common in the patch passed.
+1 💚unit1m 55shbase-client in the patch passed.
-1 ❌unit224m 22shbase-server in the patch failed.
+1 💚unit17m 0shbase-mapreduce in the patch passed.
283m 47s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 43426d559e94 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 revisionmaster / 52eef65
Default JavaEclipse Adoptium-17.0.10+7
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/testReport/
Max. process+thread count5635 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 56sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 50sMaven dependency ordering for branch
+1 💚mvninstall4m 14smaster passed
+1 💚compile1m 50smaster passed
+1 💚shadedjars7m 54sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 42smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall3m 42sthe patch passed
+1 💚compile2m 15sthe patch passed
+1 💚javac2m 15sthe patch passed
+1 💚shadedjars8m 16spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc1m 39sthe patch passed
_ Other Tests _
+1 💚unit2m 37shbase-common in the patch passed.
+1 💚unit1m 56shbase-client in the patch passed.
+1 💚unit228m 26shbase-server in the patch passed.
+1 💚unit17m 45shbase-mapreduce in the patch passed.
290m 0s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5976
Optional Testsjavac javadoc unit shadedjars compile
unameLinux ef5781e52d0c 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 revisionmaster / 52eef65
Default JavaTemurin-1.8.0_352-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/testReport/
Max. process+thread count5259 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-mapreduce U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5976/6/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9
Apache9 merged commit 85a8b54 into apache:masterJun 21, 2024
Apache9 added a commit that referenced this pull request Jun 21, 2024
Signed-off-by: Andrew Purtell <apurtell@apache.org>
(cherry picked from commit 85a8b54)
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.

3 participants

@Apache9@Apache-HBase@apurtell