Skip to content

HBASE-29835: Fix typos and inconsistencies in CacheAwareLoadBalancer - #7644

Merged
lfrancke merged 2 commits into
apache:masterfrom
lfrancke:fix/HBASE-29835
Jan 20, 2026
Merged

HBASE-29835: Fix typos and inconsistencies in CacheAwareLoadBalancer#7644
lfrancke merged 2 commits into
apache:masterfrom
lfrancke:fix/HBASE-29835

Conversation

@lfrancke

Copy link
Copy Markdown
Member

HBASE-28804 also introduced a DecimalFormat which is only used in a debug statement but instantiated once per call to a method. It is also inconsistently used.

I decided to remove it entirely in my PR. Another option would be to introduce it everywhere. I'm fine either way.

LOG.debug(
"Region {} not moved from {} to {} with current cache ratio {} and old cache ratio {}",
cluster.regions[regionIndex], cluster.servers[currentServerIndex],
cluster.servers[oldServerIndex], cacheRatioOnCurrentServer,

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Note that it would log current server ratio twice.

"Region {} moved from {} to {} as region cache ratio {} is better than the current "
+ "cache ratio {}",
cluster.regions[regionIndex].getEncodedName(), cluster.servers[currentServerIndex],
cluster.servers[oldServerIndex], cacheRatioOnCurrentServer,

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Note that it would log current ratio twice

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 21sDocker 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 _
+1 💚mvninstall2m 37smaster passed
+1 💚compile0m 11smaster passed
+1 💚javadoc0m 11smaster passed
+1 💚shadedjars4m 18sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall2m 15sthe patch passed
+1 💚compile0m 11sthe patch passed
+1 💚javac0m 11sthe patch passed
+1 💚javadoc0m 9sthe patch passed
+1 💚shadedjars4m 19spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit7m 17shbase-balancer in the patch passed.
23m 33s
SubsystemReport/Notes
DockerClientAPI=1.48 ServerAPI=1.48 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7644/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#7644
JIRA IssueHBASE-29835
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 15116cbb5f67 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 revisionmaster / 34bf291
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7644/2/testReport/
Max. process+thread count246 (vs. ulimit of 30000)
modulesC: hbase-balancer U: hbase-balancer
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7644/2/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

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 30sDocker 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.
_ master Compile Tests _
+1 💚mvninstall3m 37smaster passed
+1 💚compile0m 27smaster passed
+1 💚checkstyle0m 13smaster passed
+1 💚spotbugs0m 27smaster passed
+1 💚spotless0m 53sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall3m 10sthe patch passed
+1 💚compile0m 27sthe patch passed
+1 💚javac0m 27sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 13sthe patch passed
+1 💚spotbugs0m 31sthe patch passed
+1 💚hadoopcheck12m 20sPatch does not cause any errors with Hadoop 3.3.6 3.4.1.
+1 💚spotless0m 47spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
31m 37s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7644/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#7644
JIRA IssueHBASE-29835
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 2b9fde50dcca 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 / 34bf291
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count83 (vs. ulimit of 30000)
modulesC: hbase-balancer U: hbase-balancer
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7644/2/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.

@lfrancke
lfrancke merged commit df70b5a into apache:masterJan 20, 2026
2 checks passed
@lfrancke
lfrancke deleted the fix/HBASE-29835 branch January 20, 2026 19:27
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

@lfrancke@Apache-HBase@wchevreuil