Umeshkumar9414 commented on code in PR #7375:
URL: https://github.com/apache/hbase/pull/7375#discussion_r2515308593
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java:
##########
@@ -2114,8 +2115,8 @@ public BalanceResponse balance(BalanceRequest request)
throws IOException {
if (!request.isIgnoreRegionsInTransition() || metaInTransition) {
LOG.info("Not running balancer (ignoreRIT=false" + ", metaRIT=" +
metaInTransition
- + ") because " + regionsInTransition.size() + " region(s) in
transition: " + toPrint
- + (truncated ? "(truncated list)" : ""));
+ + ") because " + assignmentManager.getRegionTransitScheduledCount()
Review Comment:
If this question is regarding the decision making -
1. I think balancer should only run when there is no movement
expected/ongoing.
2. This was we are keeping the old logic for balancer. Before this PR
regionInTransition was reflacting the number of TRSP scheduled/active. So after
this change also I want to keep it that way
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
