refactor: cleanup variable namespace in IWRR code
authorJérôme Benoit <jerome.benoit@sap.com>
Sun, 7 May 2023 21:12:36 +0000 (23:12 +0200)
committerJérôme Benoit <jerome.benoit@sap.com>
Sun, 7 May 2023 21:12:36 +0000 (23:12 +0200)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
src/pools/selection-strategies/interleaved-weighted-round-robin-worker-choice-strategy.ts
src/pools/selection-strategies/selection-strategies-types.ts

index 24f7ac9a4cec4c151774c74b8ecaa6f50f23f02a..b14bb8bc354311b3f6539b1926763cc06705356b 100644 (file)
@@ -76,9 +76,9 @@ export class InterleavedWeightedRoundRobinWorkerChoiceStrategy<
     let roundId: number | undefined
     let workerNodeId: number | undefined
     for (
-      let round = this.currentRoundId;
-      round < this.roundWeights.length;
-      round++
+      let roundIndex = this.currentRoundId;
+      roundIndex < this.roundWeights.length;
+      roundIndex++
     ) {
       for (
         let workerNodeKey = this.currentWorkerNodeId;
@@ -87,8 +87,8 @@ export class InterleavedWeightedRoundRobinWorkerChoiceStrategy<
       ) {
         const workerWeight =
           this.opts.weights?.[workerNodeKey] ?? this.defaultWorkerWeight
-        if (workerWeight >= this.roundWeights[round]) {
-          roundId = round
+        if (workerWeight >= this.roundWeights[roundIndex]) {
+          roundId = roundIndex
           workerNodeId = workerNodeKey
           break
         }
index f2b401d6faf5bfe879225a71ab635fe4e2c625dc..6ae695ca6dca6141d48cef31dd46c6be5cb08bbb 100644 (file)
@@ -24,6 +24,8 @@ export const WorkerChoiceStrategies = Object.freeze({
   WEIGHTED_ROUND_ROBIN: 'WEIGHTED_ROUND_ROBIN',
   /**
    * Interleaved weighted round robin worker selection strategy.
+   *
+   * @experimental
    */
   INTERLEAVED_WEIGHTED_ROUND_ROBIN: 'INTERLEAVED_WEIGHTED_ROUND_ROBIN'
 } as const)