X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;ds=sidebyside;f=src%2Fpools%2Fselection-strategies%2Ffair-share-worker-choice-strategy.ts;h=58982f192ee457865e46115203b9509bdc24e29d;hb=ce63d9e2f3d895bd2ae5aafc40769ff4dda3c887;hp=f47a1e6e9ba7724fa665235a2a9204219ed2220e;hpb=776e97a2d35ba2003ddeddecc826353b569a4344;p=poolifier.git diff --git a/src/pools/selection-strategies/fair-share-worker-choice-strategy.ts b/src/pools/selection-strategies/fair-share-worker-choice-strategy.ts index f47a1e6e..58982f19 100644 --- a/src/pools/selection-strategies/fair-share-worker-choice-strategy.ts +++ b/src/pools/selection-strategies/fair-share-worker-choice-strategy.ts @@ -1,16 +1,13 @@ -import { - DEFAULT_MEASUREMENT_STATISTICS_REQUIREMENTS, - DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS -} from '../../utils' -import type { IPool } from '../pool' -import type { IWorker, StrategyData } from '../worker' -import { AbstractWorkerChoiceStrategy } from './abstract-worker-choice-strategy' +import { DEFAULT_MEASUREMENT_STATISTICS_REQUIREMENTS } from '../../utils.js' +import type { IPool } from '../pool.js' +import type { IWorker } from '../worker.js' +import { AbstractWorkerChoiceStrategy } from './abstract-worker-choice-strategy.js' import { type IWorkerChoiceStrategy, Measurements, type TaskStatisticsRequirements, type WorkerChoiceStrategyOptions -} from './selection-strategies-types' +} from './selection-strategies-types.js' /** * Selects the next worker with a fair share scheduling algorithm. @@ -45,7 +42,7 @@ export class FairShareWorkerChoiceStrategy< /** @inheritDoc */ public constructor ( pool: IPool, - opts: WorkerChoiceStrategyOptions = DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS + opts?: WorkerChoiceStrategyOptions ) { super(pool, opts) this.setTaskStatisticsRequirements(this.opts) @@ -90,9 +87,10 @@ export class FairShareWorkerChoiceStrategy< } } return this.isWorkerNodeReady(workerNodeKey) && - (workerNode.strategyData.virtualTaskEndTimestamp as number) < - ((workerNodes[minWorkerNodeKey].strategyData as StrategyData) - .virtualTaskEndTimestamp as number) + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + workerNode.strategyData.virtualTaskEndTimestamp! < + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + workerNodes[minWorkerNodeKey].strategyData!.virtualTaskEndTimestamp! ? workerNodeKey : minWorkerNodeKey }, @@ -120,7 +118,8 @@ export class FairShareWorkerChoiceStrategy< workerNodeVirtualTaskStartTimestamp: number ): number { const workerNodeTaskRunTime = - this.opts.measurement === Measurements.elu + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + this.opts!.measurement === Measurements.elu ? this.getWorkerNodeTaskElu(workerNodeKey) : this.getWorkerNodeTaskRunTime(workerNodeKey) return workerNodeVirtualTaskStartTimestamp + workerNodeTaskRunTime @@ -134,7 +133,8 @@ export class FairShareWorkerChoiceStrategy< ?.virtualTaskEndTimestamp const now = performance.now() return now < (virtualTaskEndTimestamp ?? -Infinity) - ? (virtualTaskEndTimestamp as number) + ? // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + virtualTaskEndTimestamp! : now } }