X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Fpools%2Fselection-strategies%2Fabstract-worker-choice-strategy.ts;h=d8d8f32d2f9dc2eee45f63c75f2aa0ece167b108;hb=ce63d9e2f3d895bd2ae5aafc40769ff4dda3c887;hp=91733b52c96cf1f7eff70142146965290772a3aa;hpb=ab3bb4a4fd6b09465fb97ec4f412e57448f94306;p=poolifier.git diff --git a/src/pools/selection-strategies/abstract-worker-choice-strategy.ts b/src/pools/selection-strategies/abstract-worker-choice-strategy.ts index 91733b52..d8d8f32d 100644 --- a/src/pools/selection-strategies/abstract-worker-choice-strategy.ts +++ b/src/pools/selection-strategies/abstract-worker-choice-strategy.ts @@ -1,16 +1,16 @@ import { DEFAULT_MEASUREMENT_STATISTICS_REQUIREMENTS, - buildInternalWorkerChoiceStrategyOptions -} from '../../utils' -import type { IPool } from '../pool' -import type { IWorker } from '../worker' + buildWorkerChoiceStrategyOptions +} from '../../utils.js' +import type { IPool } from '../pool.js' +import type { IWorker } from '../worker.js' import type { IWorkerChoiceStrategy, - InternalWorkerChoiceStrategyOptions, MeasurementStatisticsRequirements, StrategyPolicy, - TaskStatisticsRequirements -} from './selection-strategies-types' + TaskStatisticsRequirements, + WorkerChoiceStrategyOptions +} from './selection-strategies-types.js' /** * Worker choice strategy abstract base class. @@ -55,26 +55,29 @@ export abstract class AbstractWorkerChoiceStrategy< */ public constructor ( protected readonly pool: IPool, - protected opts: InternalWorkerChoiceStrategyOptions + protected opts?: WorkerChoiceStrategyOptions ) { this.setOptions(this.opts) this.choose = this.choose.bind(this) } protected setTaskStatisticsRequirements ( - opts: InternalWorkerChoiceStrategyOptions + opts: WorkerChoiceStrategyOptions | undefined ): void { this.toggleMedianMeasurementStatisticsRequirements( this.taskStatisticsRequirements.runTime, - opts.runTime?.median as boolean + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + opts!.runTime!.median ) this.toggleMedianMeasurementStatisticsRequirements( this.taskStatisticsRequirements.waitTime, - opts.waitTime?.median as boolean + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + opts!.waitTime!.median ) this.toggleMedianMeasurementStatisticsRequirements( this.taskStatisticsRequirements.elu, - opts.elu?.median as boolean + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + opts!.elu!.median ) } @@ -110,9 +113,9 @@ export abstract class AbstractWorkerChoiceStrategy< public abstract remove (workerNodeKey: number): boolean /** @inheritDoc */ - public setOptions (opts: InternalWorkerChoiceStrategyOptions): void { - this.opts = buildInternalWorkerChoiceStrategyOptions( - this.pool.info.maxSize, + public setOptions (opts: WorkerChoiceStrategyOptions | undefined): void { + this.opts = buildWorkerChoiceStrategyOptions( + this.pool, opts ) this.setTaskStatisticsRequirements(this.opts) @@ -137,7 +140,8 @@ export abstract class AbstractWorkerChoiceStrategy< * Check the next worker node readiness. */ protected checkNextWorkerNodeReadiness (): void { - if (!this.isWorkerNodeReady(this.nextWorkerNodeKey as number)) { + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + if (!this.isWorkerNodeReady(this.nextWorkerNodeKey!)) { delete this.nextWorkerNodeKey } }