(backport v1.57) removed assert statement for static stride scheduler…

… with multiple threads (#10438)
grpc · Jul 31, 2023 · 38e0a0f · 38e0a0f
1 parent 503883b
commit 38e0a0f
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/xds/src/main/java/io/grpc/xds/WeightedRoundRobinLoadBalancer.java b/xds/src/main/java/io/grpc/xds/WeightedRoundRobinLoadBalancer.java
@@ -431,9 +431,7 @@ long getSequence() {
      * an offset that varies per backend index is also included to the calculation.
      */
     int pick() {
-      int i = 0;
       while (true) {
-        i++;
         long sequence = this.nextSequence();
         int backendIndex = (int) (sequence % scaledWeights.length);
         long generation = sequence / scaledWeights.length;
@@ -442,7 +440,6 @@ int pick() {
         if ((weight * generation + offset) % K_MAX_WEIGHT < K_MAX_WEIGHT - weight) {
           continue;
         }
-        assert i <= scaledWeights.length : "scheduler has more than one pass through";
         return backendIndex;
       }
     }