X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Fpools%2Fselection-strategies%2Fabstract-worker-choice-strategy.ts;h=ce7f9589ab5708975019706047b477f71007731f;hb=95311547b3ec8a54b9e4303bdd3c2c32c9c1e893;hp=31041f3a8783c605d237196f7e89920d04dd27bb;hpb=da3098610d6cf6155bbbe53c84b6ac6ccd400ff5;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 31041f3a..ce7f9589 100644 --- a/src/pools/selection-strategies/abstract-worker-choice-strategy.ts +++ b/src/pools/selection-strategies/abstract-worker-choice-strategy.ts @@ -1,5 +1,5 @@ -import type { IPoolInternal } from '../pool-internal' -import { PoolType } from '../pool-internal' +import { DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS } from '../../utils' +import { PoolType, type IPool } from '../pool' import type { IWorker } from '../worker' import type { IWorkerChoiceStrategy, @@ -12,7 +12,7 @@ import type { * * @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 Response - Type of execution response. This can only be serializable data. */ export abstract class AbstractWorkerChoiceStrategy< Worker extends IWorker, @@ -35,16 +35,16 @@ export abstract class AbstractWorkerChoiceStrategy< * @param opts - The worker choice strategy options. */ public constructor ( - protected readonly pool: IPoolInternal, - protected readonly opts: WorkerChoiceStrategyOptions = { medRunTime: false } + protected readonly pool: IPool, + protected readonly opts: WorkerChoiceStrategyOptions = DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS ) { - this.checkOptions() + this.checkOptions(this.opts) this.isDynamicPool = this.pool.type === PoolType.DYNAMIC this.choose.bind(this) } - private checkOptions (): void { - if (this.requiredStatistics.avgRunTime && this.opts.medRunTime === true) { + private checkOptions (opts: WorkerChoiceStrategyOptions): void { + if (this.requiredStatistics.avgRunTime && opts.medRunTime === true) { this.requiredStatistics.medRunTime = true } }