-import type { IPoolWorker } from '../pool-worker'
+import type { IPool } from '../pool'
+import type { IWorker } from '../worker'
import { AbstractWorkerChoiceStrategy } from './abstract-worker-choice-strategy'
-import type { IWorkerChoiceStrategy } from './selection-strategies-types'
+import type {
+ IWorkerChoiceStrategy,
+ InternalWorkerChoiceStrategyOptions
+} from './selection-strategies-types'
/**
* Selects the next worker in a round robin fashion.
*
* @typeParam Worker - Type of worker which manages the strategy.
- * @typeParam Data - Type of data sent to the worker. This can only be serializable data.
- * @typeParam Response - Type of response of execution. This can only be serializable data.
+ * @typeParam Data - Type of data sent to the worker. This can only be structured-cloneable data.
+ * @typeParam Response - Type of execution response. This can only be structured-cloneable data.
*/
export class RoundRobinWorkerChoiceStrategy<
- Worker extends IPoolWorker,
+ Worker extends IWorker,
Data = unknown,
Response = unknown
>
extends AbstractWorkerChoiceStrategy<Worker, Data, Response>
implements IWorkerChoiceStrategy {
- /**
- * Id of the next worker.
- */
- private nextWorkerId: number = 0
+ /** @inheritDoc */
+ public constructor (
+ pool: IPool<Worker, Data, Response>,
+ opts: InternalWorkerChoiceStrategyOptions
+ ) {
+ super(pool, opts)
+ }
/** @inheritDoc */
public reset (): boolean {
- this.nextWorkerId = 0
+ this.resetWorkerNodeKeyProperties()
return true
}
/** @inheritDoc */
- public choose (): number {
- const chosenWorkerKey = this.nextWorkerId
- this.nextWorkerId =
- this.nextWorkerId === this.pool.workers.length - 1
- ? 0
- : this.nextWorkerId + 1
- return chosenWorkerKey
+ public update (): boolean {
+ return true
+ }
+
+ /** @inheritDoc */
+ public choose (): number | undefined {
+ const chosenWorkerNodeKey = this.nextWorkerNodeKey
+ this.setPreviousWorkerNodeKey(chosenWorkerNodeKey)
+ this.roundRobinNextWorkerNodeKey()
+ this.checkNextWorkerNodeReadiness()
+ return chosenWorkerNodeKey
}
/** @inheritDoc */
- public remove (workerKey: number): boolean {
- if (this.nextWorkerId === workerKey) {
- if (this.pool.workers.length === 0) {
- this.nextWorkerId = 0
- } else {
- this.nextWorkerId =
- this.nextWorkerId > this.pool.workers.length - 1
- ? this.pool.workers.length - 1
- : this.nextWorkerId
- }
+ public remove (workerNodeKey: number): boolean {
+ if (this.pool.workerNodes.length === 0) {
+ this.reset()
+ }
+ if (
+ this.nextWorkerNodeKey === workerNodeKey &&
+ this.nextWorkerNodeKey > this.pool.workerNodes.length - 1
+ ) {
+ this.nextWorkerNodeKey = this.pool.workerNodes.length - 1
+ }
+ if (
+ this.previousWorkerNodeKey === workerNodeKey &&
+ this.previousWorkerNodeKey > this.pool.workerNodes.length - 1
+ ) {
+ this.previousWorkerNodeKey = this.pool.workerNodes.length - 1
}
return true
}
+
+ private roundRobinNextWorkerNodeKey (): number | undefined {
+ this.nextWorkerNodeKey =
+ this.nextWorkerNodeKey === this.pool.workerNodes.length - 1
+ ? 0
+ : (this.nextWorkerNodeKey ?? this.previousWorkerNodeKey) + 1
+ return this.nextWorkerNodeKey
+ }
}