Skip to content

Backport "HBASE-29663 TimeBasedLimiters should support dynamic configuration re…" to branch-2 - #7393

Merged
rmdmattingly merged 1 commit into
branch-2from
HBASE-29663-branch-2
Oct 17, 2025
Merged

Backport "HBASE-29663 TimeBasedLimiters should support dynamic configuration re…" to branch-2#7393
rmdmattingly merged 1 commit into
branch-2from
HBASE-29663-branch-2

Conversation

@rmdmattingly

@rmdmattinglyrmdmattingly commented Oct 16, 2025

Copy link
Copy Markdown
Contributor

…fresh (#7387)

@rmdmattinglyRay Mattingly
HBASE-29663TimeBasedLimiters should support dynamic configuration re…
a6edd64
…fresh (#7387)
Co-authored-by: Ray Mattingly <rmattingly@hubspot.com>
Signed-off-by: Charles Connell <cconnell@apache.org>
Signed-off-by: Nick Dimiduk <ndimiduk@apache.org>
@Apache-HBase

Apache-HBase commented Oct 16, 2025

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 4sDocker 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-2 Compile Tests _
+1 💚mvninstall4m 50sbranch-2 passed
+1 💚compile3m 22sbranch-2 passed
+1 💚checkstyle0m 50sbranch-2 passed
+1 💚spotbugs2m 27sbranch-2 passed
+1 💚spotless0m 51sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall3m 14sthe patch passed
+1 💚compile3m 48sthe patch passed
+1 💚javac3m 48sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle1m 13sthe patch passed
+1 💚spotbugs3m 7sthe patch passed
+1 💚hadoopcheck19m 16sPatch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.1.
+1 💚spotless0m 44spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 11sThe patch does not generate ASF License warnings.
47m 23s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#7393
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 1b42c77c2074 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-2 / a6edd64
Default JavaEclipse Adoptium-11.0.23+9
Max. process+thread count77 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/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

Apache-HBase commented Oct 16, 2025

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec2m 20sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+1 💚mvninstall4m 59sbranch-2 passed
+1 💚compile1m 2sbranch-2 passed
+1 💚javadoc0m 39sbranch-2 passed
+1 💚shadedjars7m 37sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall4m 6sthe patch passed
+1 💚compile1m 1sthe patch passed
+1 💚javac1m 1sthe patch passed
+1 💚javadoc0m 35sthe patch passed
+1 💚shadedjars7m 24spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
-1 ❌unit493m 55s/patch-unit-hbase-server.txthbase-server in the patch failed.
528m 14s
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#7393
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 17590e13078e 6.8.0-1024-aws #26~22.04.1-Ubuntu SMP Wed Feb 19 06:54:57 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / a6edd64
Default JavaTemurin-1.8.0_412-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/1/testReport/
Max. process+thread count2213 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/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.

@Apache-HBase

Apache-HBase commented Oct 16, 2025

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec3m 0sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+1 💚mvninstall6m 9sbranch-2 passed
+1 💚compile1m 59sbranch-2 passed
+1 💚javadoc1m 5sbranch-2 passed
+1 💚shadedjars10m 35sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall4m 50sthe patch passed
+1 💚compile1m 23sthe patch passed
+1 💚javac1m 23sthe patch passed
+1 💚javadoc0m 39sthe patch passed
+1 💚shadedjars8m 42spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
-1 ❌unit540m 5s/patch-unit-hbase-server.txthbase-server in the patch failed.
583m 12s
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#7393
Optional Testsjavac javadoc unit compile shadedjars
unameLinux de26be106b29 6.8.0-1024-aws #26~22.04.1-Ubuntu SMP Wed Feb 19 06:54:57 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / a6edd64
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/1/testReport/
Max. process+thread count2376 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/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.

@Apache-HBase

Apache-HBase commented Oct 16, 2025

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec2m 16sDocker mode activated.
-0 ⚠️yetus0m 6sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+1 💚mvninstall6m 11sbranch-2 passed
+1 💚compile1m 18sbranch-2 passed
+1 💚javadoc0m 43sbranch-2 passed
+1 💚shadedjars10m 9sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall5m 39sthe patch passed
+1 💚compile1m 27sthe patch passed
+1 💚javac1m 27sthe patch passed
+1 💚javadoc0m 45sthe patch passed
+1 💚shadedjars8m 55spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
-1 ❌unit543m 21s/patch-unit-hbase-server.txthbase-server in the patch failed.
585m 30s
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#7393
Optional Testsjavac javadoc unit compile shadedjars
unameLinux e8a17865e3e7 6.8.0-1024-aws #26~22.04.1-Ubuntu SMP Wed Feb 19 06:54:57 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / a6edd64
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/1/testReport/
Max. process+thread count2390 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7393/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.

@rmdmattingly
rmdmattingly merged commit 69e7dcc into branch-2Oct 17, 2025
1 check failed
@rmdmattingly
rmdmattingly deleted the HBASE-29663-branch-2 branch October 17, 2025 13:01
rmdmattingly added a commit that referenced this pull request Oct 17, 2025
@rmdmattinglyRay Mattingly
HBASE-29663TimeBasedLimiters should support dynamic configuration re…
d16471e
…fresh (#7387) (#7393)
Signed-off-by: Charles Connell <cconnell@apache.org>
Signed-off-by: Nick Dimiduk <ndimiduk@apache.org>
Co-authored-by: Ray Mattingly <rmattingly@hubspot.com>
@rmdmattinglyrmdmattingly mentioned this pull request Oct 17, 2025
Merged
rmdmattingly added a commit that referenced this pull request Oct 17, 2025
@rmdmattinglyRay Mattingly
HBASE-29663TimeBasedLimiters should support dynamic configuration re…
26cbc3e
…fresh (#7387) (#7393) (#7395)
Signed-off-by: Charles Connell <cconnell@apache.org>
Signed-off-by: Nick Dimiduk <ndimiduk@apache.org>
Co-authored-by: Ray Mattingly <rmattingly@hubspot.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rmdmattingly@Apache-HBase