Refactor hasComputationConverged (#116331)

See
#115511 (comment).

Author: pxsalehi

server/src/main/java/org/elasticsearch/cluster/routing/allocation/allocator/DesiredBalanceComputer.java

@@ -335,7 +335,7 @@ public DesiredBalance compute(
                 nextReportTime = currentTime + timeWarningInterval;
             }
 
-            if (hasComputationConverged(hasChanges, i)) {
+            if (hasChanges == false && hasEnoughIterations(i)) {
                 logger.debug(
                     "Desired balance computation for [{}] converged after [{}] and [{}] iterations",
                     desiredBalanceInput.index(),
@@ -415,8 +415,8 @@ public DesiredBalance compute(
     }
 
     // visible for testing
-    boolean hasComputationConverged(boolean hasRoutingChanges, int currentIteration) {
-        return hasRoutingChanges == false;
+    boolean hasEnoughIterations(int currentIteration) {
+        return true;
     }
 
     private static Map<ShardId, ShardAssignment> collectShardAssignments(RoutingNodes routingNodes) {

server/src/test/java/org/elasticsearch/cluster/routing/allocation/allocator/DesiredBalanceShardsAllocatorTests.java

@@ -408,8 +408,8 @@ public DesiredBalance compute(
                 }
 
                 @Override
-                boolean hasComputationConverged(boolean hasRoutingChanges, int currentIteration) {
-                    return super.hasComputationConverged(hasRoutingChanges, currentIteration) && currentIteration >= minIterations;
+                boolean hasEnoughIterations(int currentIteration) {
+                    return currentIteration >= minIterations;
                 }
             },
             reconcileAction,