repositories
/
poolifier.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
refactor: forEach -> for ... of on collections
[poolifier.git]
/
src
/
pools
/
selection-strategies
/
worker-choice-strategy-context.ts
diff --git
a/src/pools/selection-strategies/worker-choice-strategy-context.ts
b/src/pools/selection-strategies/worker-choice-strategy-context.ts
index 0a488549f9f9d7240b8d4fd421d64b89ada5401b..e07808bcefe837ef5289be98db46ac3664607219 100644
(file)
--- a/
src/pools/selection-strategies/worker-choice-strategy-context.ts
+++ b/
src/pools/selection-strategies/worker-choice-strategy-context.ts
@@
-115,33
+115,38
@@
export class WorkerChoiceStrategyContext<
}
/**
}
/**
- * Updates the worker choice strategy internals in the context.
+ * Updates the worker
node key in the worker
choice strategy internals in the context.
*
* @returns `true` if the update is successful, `false` otherwise.
*/
*
* @returns `true` if the update is successful, `false` otherwise.
*/
- public update (): boolean {
+ public update (
workerNodeKey: number
): boolean {
return (
this.workerChoiceStrategies.get(
this.workerChoiceStrategy
) as IWorkerChoiceStrategy
return (
this.workerChoiceStrategies.get(
this.workerChoiceStrategy
) as IWorkerChoiceStrategy
- ).update()
+ ).update(
workerNodeKey
)
}
/**
* Executes the worker choice strategy algorithm in the context.
*
* @returns The key of the worker node.
}
/**
* Executes the worker choice strategy algorithm in the context.
*
* @returns The key of the worker node.
+ * @throws {@link https://nodejs.org/api/errors.html#class-error} If the worker node key is null or undefined.
*/
public execute (): number {
*/
public execute (): number {
-
return
(
+
const workerNodeKey =
(
this.workerChoiceStrategies.get(
this.workerChoiceStrategy
) as IWorkerChoiceStrategy
).choose()
this.workerChoiceStrategies.get(
this.workerChoiceStrategy
) as IWorkerChoiceStrategy
).choose()
+ if (workerNodeKey == null) {
+ throw new Error('Worker node key chosen is null or undefined')
+ }
+ return workerNodeKey
}
/**
}
/**
- * Removes
a
worker node key from the worker choice strategy in the context.
+ * Removes
the
worker node key from the worker choice strategy in the context.
*
* @param workerNodeKey - The key of the worker node.
* @returns `true` if the removal is successful, `false` otherwise.
*
* @param workerNodeKey - The key of the worker node.
* @returns `true` if the removal is successful, `false` otherwise.
@@
-160,8
+165,8
@@
export class WorkerChoiceStrategyContext<
* @param opts - The worker choice strategy options.
*/
public setOptions (opts: WorkerChoiceStrategyOptions): void {
* @param opts - The worker choice strategy options.
*/
public setOptions (opts: WorkerChoiceStrategyOptions): void {
-
this.workerChoiceStrategies.forEach(workerChoiceStrategy =>
{
+
for (const workerChoiceStrategy of this.workerChoiceStrategies.values())
{
workerChoiceStrategy.setOptions(opts)
workerChoiceStrategy.setOptions(opts)
- }
)
+ }
}
}
}
}