keith-turner commented on code in PR #5596:
URL: https://github.com/apache/accumulo/pull/5596#discussion_r2114676785
##########
test/src/main/java/org/apache/accumulo/test/ChaoticLoadBalancer.java:
##########
@@ -66,37 +67,23 @@ public void init(BalancerEnvironment balancerEnvironment) {
@Override
public void getAssignments(AssignmentParameters params) {
- long total = params.unassignedTablets().size();
- long avg = (long) Math.ceil(((double) total) /
params.currentStatus().size());
- Map<TabletServerId,Long> toAssign = new HashMap<>();
- List<TabletServerId> tServerArray = new ArrayList<>();
- for (Entry<TabletServerId,TServerStatus> e :
params.currentStatus().entrySet()) {
- long numTablets = 0;
- for (TableStatistics ti : e.getValue().getTableMap().values()) {
- numTablets += ti.getTabletCount();
- }
- if (numTablets <= avg) {
Review Comment:
When tservers had other user and metadata tablets, this would always be
false for the root tablet and a tserver would never be added.
--
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
