import crypto from 'node:crypto'
import type { MessageValue, PromiseResponseWrapper } from '../utility-types'
-import { EMPTY_FUNCTION, median } from '../utils'
+import {
+ DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS,
+ EMPTY_FUNCTION,
+ median
+} from '../utils'
import { KillBehaviors, isKillBehavior } from '../worker/worker-options'
import { PoolEvents, type PoolOptions } from './pool'
import { PoolEmitter } from './pool'
this.checkPoolOptions(this.opts)
this.chooseWorkerNode.bind(this)
- this.internalExecute.bind(this)
+ this.executeTask.bind(this)
+ this.enqueueTask.bind(this)
this.checkAndEmitEvents.bind(this)
- this.sendToWorker.bind(this)
this.setupHook()
opts.workerChoiceStrategy ?? WorkerChoiceStrategies.ROUND_ROBIN
this.checkValidWorkerChoiceStrategy(this.opts.workerChoiceStrategy)
this.opts.workerChoiceStrategyOptions =
- opts.workerChoiceStrategyOptions ?? { medRunTime: false }
+ opts.workerChoiceStrategyOptions ?? DEFAULT_WORKER_CHOICE_STRATEGY_OPTIONS
this.opts.enableEvents = opts.enableEvents ?? true
this.opts.enableTasksQueue = opts.enableTasksQueue ?? false
}
public abstract get busy (): boolean
protected internalBusy (): boolean {
- return (
- this.numberOfRunningTasks >= this.numberOfWorkers &&
- this.findFreeWorkerNodeKey() === -1
- )
+ return this.findFreeWorkerNodeKey() === -1
}
/** @inheritDoc */
data: data ?? ({} as Data),
id: crypto.randomUUID()
}
- const res = this.internalExecute(workerNodeKey, workerNode, submittedTask)
- let currentTask: Task<Data> = submittedTask
+ const res = new Promise<Response>((resolve, reject) => {
+ this.promiseResponseMap.set(submittedTask.id, {
+ resolve,
+ reject,
+ worker: workerNode.worker
+ })
+ })
if (
this.opts.enableTasksQueue === true &&
- (this.busy || this.tasksQueueSize(workerNodeKey) > 0)
+ (this.busy || this.workerNodes[workerNodeKey].tasksUsage.running > 0)
) {
- currentTask = this.enqueueDequeueTask(
- workerNodeKey,
- submittedTask
- ) as Task<Data>
+ this.enqueueTask(workerNodeKey, submittedTask)
+ } else {
+ this.executeTask(workerNodeKey, submittedTask)
}
- this.sendToWorker(workerNode.worker, currentTask)
this.checkAndEmitEvents()
// eslint-disable-next-line @typescript-eslint/return-await
return res
/** @inheritDoc */
public async destroy (): Promise<void> {
await Promise.all(
- this.workerNodes.map(async workerNode => {
- this.flushTasksQueueByWorker(workerNode.worker)
+ this.workerNodes.map(async (workerNode, workerNodeKey) => {
+ this.flushTasksQueue(workerNodeKey)
await this.destroyWorker(workerNode.worker)
})
)
protected abstract destroyWorker (worker: Worker): void | Promise<void>
/**
- * Setup hook to run code before worker node are created in the abstract constructor.
+ * Setup hook to execute code before worker node are created in the abstract constructor.
* Can be overridden
*
* @virtual
protected abstract isMain (): boolean
/**
- * Hook executed before the worker task promise resolution.
+ * Hook executed before the worker task execution.
* Can be overridden.
*
* @param workerNodeKey - The worker node key.
*/
- protected beforePromiseResponseHook (workerNodeKey: number): void {
+ protected beforeTaskExecutionHook (workerNodeKey: number): void {
++this.workerNodes[workerNodeKey].tasksUsage.running
}
/**
- * Hook executed after the worker task promise resolution.
+ * Hook executed after the worker task execution.
* Can be overridden.
*
* @param worker - The worker.
* @param message - The received message.
*/
- protected afterPromiseResponseHook (
+ protected afterTaskExecutionHook (
worker: Worker,
message: MessageValue<Response>
): void {
*/
protected chooseWorkerNode (): [number, WorkerNode<Worker, Data>] {
let workerNodeKey: number
- if (
- this.type === PoolType.DYNAMIC &&
- !this.full &&
- this.findFreeWorkerNodeKey() === -1
- ) {
+ if (this.type === PoolType.DYNAMIC && !this.full && this.internalBusy()) {
const workerCreated = this.createAndSetupWorker()
this.registerWorkerMessageListener(workerCreated, message => {
if (
} else {
promiseResponse.resolve(message.data as Response)
}
- this.afterPromiseResponseHook(promiseResponse.worker, message)
+ this.afterTaskExecutionHook(promiseResponse.worker, message)
this.promiseResponseMap.delete(message.id)
const workerNodeKey = this.getWorkerNodeKey(promiseResponse.worker)
if (
this.opts.enableTasksQueue === true &&
this.tasksQueueSize(workerNodeKey) > 0
) {
- this.sendToWorker(
- promiseResponse.worker,
+ this.executeTask(
+ workerNodeKey,
this.dequeueTask(workerNodeKey) as Task<Data>
)
}
}
}
- private async internalExecute (
- workerNodeKey: number,
- workerNode: WorkerNode<Worker, Data>,
- task: Task<Data>
- ): Promise<Response> {
- this.beforePromiseResponseHook(workerNodeKey)
- return await new Promise<Response>((resolve, reject) => {
- this.promiseResponseMap.set(task.id, {
- resolve,
- reject,
- worker: workerNode.worker
- })
- })
- }
-
private checkAndEmitEvents (): void {
if (this.opts.enableEvents === true) {
if (this.busy) {
this.workerChoiceStrategyContext.remove(workerNodeKey)
}
- private enqueueDequeueTask (
- workerNodeKey: number,
- task: Task<Data>
- ): Task<Data> | undefined {
- this.enqueueTask(workerNodeKey, task)
- return this.dequeueTask(workerNodeKey)
+ private executeTask (workerNodeKey: number, task: Task<Data>): void {
+ this.beforeTaskExecutionHook(workerNodeKey)
+ this.sendToWorker(this.workerNodes[workerNodeKey].worker, task)
}
- private enqueueTask (workerNodeKey: number, task: Task<Data>): void {
- this.workerNodes[workerNodeKey].tasksQueue.push(task)
+ private enqueueTask (workerNodeKey: number, task: Task<Data>): number {
+ return this.workerNodes[workerNodeKey].tasksQueue.push(task)
}
private dequeueTask (workerNodeKey: number): Task<Data> | undefined {
private flushTasksQueue (workerNodeKey: number): void {
if (this.tasksQueueSize(workerNodeKey) > 0) {
for (const task of this.workerNodes[workerNodeKey].tasksQueue) {
- this.sendToWorker(this.workerNodes[workerNodeKey].worker, task)
+ this.executeTask(workerNodeKey, task)
}
- this.workerNodes[workerNodeKey].tasksQueue = []
}
}