EdColeman commented on code in PR #4523:
URL: https://github.com/apache/accumulo/pull/4523#discussion_r1589594441
##########
core/src/main/java/org/apache/accumulo/core/spi/balancer/HostRegexTableLoadBalancer.java:
##########
@@ -466,6 +475,13 @@ public long balance(BalanceParameters params) {
LOG.trace("Sample up to 10 outstanding migrations: {}",
limitTen(migrations));
}
return minBalanceTime;
+ } else if (!myConf.concurrentTables) {
+ LOG.warn("Not balancing tables due to {} existing migrations and {}}
is set to false",
Review Comment:
This seems another case where we really could benefit from being able to log
1 of Number of 1 every T seconds occurrences of the same message. It is not
helpful to keep repeating that same message frequently, but having it appear
occasionally helps troubleshoot cases where something in not making progress -
might be out of scope for this, but could be worthwhile.
--
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: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
