-import { DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS } from '../../utils'
+import {
+ DEFAULT_MEASUREMENT_STATISTICS_REQUIREMENTS,
+ DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS
+} from '../../utils'
import type { IPool } from '../pool'
import type { IWorker } from '../worker'
import { AbstractWorkerChoiceStrategy } from './abstract-worker-choice-strategy'
* Loosely modeled after the fair queueing algorithm: https://en.wikipedia.org/wiki/Fair_queuing.
*
* @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 execution response. 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 FairShareWorkerChoiceStrategy<
Worker extends IWorker,
average: true,
median: false
},
- waitTime: {
- aggregate: false,
- average: false,
- median: false
- },
+ waitTime: DEFAULT_MEASUREMENT_STATISTICS_REQUIREMENTS,
elu: {
aggregate: true,
average: true,
/** @inheritDoc */
public choose (): number {
+ return this.fairShareNextWorkerNodeKey()
+ }
+
+ /** @inheritDoc */
+ public remove (workerNodeKey: number): boolean {
+ this.workersVirtualTaskEndTimestamp.splice(workerNodeKey, 1)
+ return true
+ }
+
+ private fairShareNextWorkerNodeKey (): number {
let minWorkerVirtualTaskEndTimestamp = Infinity
- let chosenWorkerNodeKey!: number
for (const [workerNodeKey] of this.pool.workerNodes.entries()) {
if (this.workersVirtualTaskEndTimestamp[workerNodeKey] == null) {
this.computeWorkerVirtualTaskEndTimestamp(workerNodeKey)
}
const workerVirtualTaskEndTimestamp =
this.workersVirtualTaskEndTimestamp[workerNodeKey]
- if (workerVirtualTaskEndTimestamp < minWorkerVirtualTaskEndTimestamp) {
+ if (
+ this.isWorkerNodeReady(workerNodeKey) &&
+ workerVirtualTaskEndTimestamp < minWorkerVirtualTaskEndTimestamp
+ ) {
minWorkerVirtualTaskEndTimestamp = workerVirtualTaskEndTimestamp
- chosenWorkerNodeKey = workerNodeKey
+ this.nextWorkerNodeKey = workerNodeKey
}
}
- return chosenWorkerNodeKey
- }
-
- /** @inheritDoc */
- public remove (workerNodeKey: number): boolean {
- this.workersVirtualTaskEndTimestamp.splice(workerNodeKey, 1)
- return true
+ return this.nextWorkerNodeKey
}
/**