Skip to content

HBASE-28862 Change the generic type for ObserverContext from 'RegionC… - #6272

Merged
Apache9 merged 1 commit into
apache:masterfrom
Apache9:HBASE-28862
Oct 10, 2024
Merged

HBASE-28862 Change the generic type for ObserverContext from 'RegionC…#6272
Apache9 merged 1 commit into
apache:masterfrom
Apache9:HBASE-28862

Conversation

@Apache9

Copy link
Copy Markdown
Contributor

…oprocessorEnvironment' to '? extends RegionCoprocessorEnvironment' in RegionObserver

…oprocessorEnvironment' to '? extends RegionCoprocessorEnvironment' in RegionObserver
@Apache9Apache9 self-assigned this Sep 19, 2024
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 39sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo 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.
_ master Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall3m 2smaster passed
+1 💚compile5m 57smaster passed
+1 💚checkstyle1m 50smaster passed
+1 💚spotbugs4m 52smaster passed
+1 💚spotless0m 47sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 9sMaven dependency ordering for patch
+1 💚mvninstall2m 56sthe patch passed
+1 💚compile5m 59sthe patch passed
+1 💚javac5m 59sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle1m 48sthe patch passed
+1 💚spotbugs5m 36sthe patch passed
+1 💚hadoopcheck10m 24sPatch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚spotless0m 44spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 59sThe patch does not generate ASF License warnings.
53m 47s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6272/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6272
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux dbb57d01b33b 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 3b0cfbc
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count86 (vs. ulimit of 30000)
modulesC: hbase-server hbase-mapreduce hbase-thrift hbase-backup hbase-it hbase-examples U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6272/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 🆗reexec1m 1sDocker 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 _
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall4m 32smaster passed
+1 💚compile4m 21smaster passed
+1 💚javadoc3m 17smaster passed
+1 💚shadedjars7m 47sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall4m 21sthe patch passed
+1 💚compile3m 38sthe patch passed
+1 💚javac3m 38sthe patch passed
+1 💚javadoc2m 15sthe patch passed
+1 💚shadedjars6m 33spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit233m 46shbase-server in the patch passed.
+1 💚unit30m 33shbase-mapreduce in the patch passed.
+1 💚unit7m 33shbase-thrift in the patch passed.
+1 💚unit13m 38shbase-backup in the patch passed.
+1 💚unit1m 30shbase-it in the patch passed.
+1 💚unit3m 47shbase-examples in the patch passed.
335m 15s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6272/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6272
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 695912fa2356 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 3b0cfbc
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6272/1/testReport/
Max. process+thread count4776 (vs. ulimit of 30000)
modulesC: hbase-server hbase-mapreduce hbase-thrift hbase-backup hbase-it hbase-examples U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6272/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.

@ndimidukndimiduk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What a beast of refactoring commit. Thanks @Apache9 .

@Apache9
Apache9 merged commit 402aa92 into apache:masterOct 10, 2024
Apache9 added a commit that referenced this pull request Oct 10, 2024
…oprocessorEnvironment' to '? extends RegionCoprocessorEnvironment' in RegionObserver (#6272)
Signed-off-by: Nick Dimiduk <ndimiduk@apache.org>
(cherry picked from commit 402aa92)
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@ndimiduk