X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Fpools%2Fworker-node.ts;h=73b2c70c3f3b05ce1031bf57fe5522c40874cfa6;hb=332b2df7b41be1f5705346612d5e8b83f7fd1ac3;hp=7a6a1500f962e4cc948ad593f3f40e45d49e53ad;hpb=07e0c9e591f9fa6715ba94e52c647b7ee3d2b9c7;p=poolifier.git diff --git a/src/pools/worker-node.ts b/src/pools/worker-node.ts index 7a6a1500..73b2c70c 100644 --- a/src/pools/worker-node.ts +++ b/src/pools/worker-node.ts @@ -1,24 +1,21 @@ import { MessageChannel } from 'node:worker_threads' import { EventEmitter } from 'node:events' -import { CircularArray } from '../circular-array' -import type { Task } from '../utility-types' -import { DEFAULT_TASK_NAME, getWorkerId, getWorkerType } from '../utils' -import { Deque } from '../deque' +import { CircularArray } from '../circular-array.js' +import type { Task } from '../utility-types.js' +import { DEFAULT_TASK_NAME, getWorkerId, getWorkerType } from '../utils.js' +import { Deque } from '../deque.js' import { - type ErrorHandler, - type ExitHandler, + type EventHandler, type IWorker, type IWorkerNode, - type MessageHandler, - type OnlineHandler, type StrategyData, type WorkerInfo, type WorkerNodeOptions, type WorkerType, WorkerTypes, type WorkerUsage -} from './worker' -import { checkWorkerNodeArguments, createWorker } from './utils' +} from './worker.js' +import { checkWorkerNodeArguments, createWorker } from './utils.js' /** * Worker node. @@ -49,7 +46,7 @@ export class WorkerNode * Constructs a new worker node. * * @param type - The worker type. - * @param filePath - The worker file path. + * @param filePath - Path to the worker file. * @param opts - The worker node options. */ constructor (type: WorkerType, filePath: string, opts: WorkerNodeOptions) { @@ -80,7 +77,8 @@ export class WorkerNode const tasksQueueSize = this.tasksQueue.push(task) if (this.hasBackPressure() && !this.onBackPressureStarted) { this.onBackPressureStarted = true - this.emit('backPressure', { workerId: this.info.id as number }) + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + this.emit('backPressure', { workerId: this.info.id! }) this.onBackPressureStarted = false } return tasksQueueSize @@ -91,7 +89,8 @@ export class WorkerNode const tasksQueueSize = this.tasksQueue.unshift(task) if (this.hasBackPressure() && !this.onBackPressureStarted) { this.onBackPressureStarted = true - this.emit('backPressure', { workerId: this.info.id as number }) + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + this.emit('backPressure', { workerId: this.info.id! }) this.onBackPressureStarted = false } return tasksQueueSize @@ -132,13 +131,16 @@ export class WorkerNode }) this.closeMessageChannel() this.removeAllListeners() - if (this.info.type === WorkerTypes.thread) { - await this.worker.terminate?.() - } else if (this.info.type === WorkerTypes.cluster) { - this.registerOnceWorkerEventHandler('disconnect', () => { - this.worker.kill?.() - }) - this.worker.disconnect?.() + switch (this.info.type) { + case WorkerTypes.thread: + await this.worker.terminate?.() + break + case WorkerTypes.cluster: + this.registerOnceWorkerEventHandler('disconnect', () => { + this.worker.kill?.() + }) + this.worker.disconnect?.() + break } await waitWorkerExit } @@ -146,25 +148,17 @@ export class WorkerNode /** @inheritdoc */ public registerWorkerEventHandler ( event: string, - listener: - | OnlineHandler - | MessageHandler - | ErrorHandler - | ExitHandler + handler: EventHandler ): void { - this.worker.on(event, listener) + this.worker.on(event, handler) } /** @inheritdoc */ public registerOnceWorkerEventHandler ( event: string, - listener: - | OnlineHandler - | MessageHandler - | ErrorHandler - | ExitHandler + handler: EventHandler ): void { - this.worker.once(event, listener) + this.worker.once(event, handler) } /** @inheritdoc */ @@ -209,9 +203,11 @@ export class WorkerNode private initWorkerInfo (worker: Worker): WorkerInfo { return { id: getWorkerId(worker), - type: getWorkerType(worker) as WorkerType, + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + type: getWorkerType(worker)!, dynamic: false, - ready: false + ready: false, + stealing: false } } @@ -259,7 +255,8 @@ export class WorkerNode for (const task of this.tasksQueue) { if ( (task.name === DEFAULT_TASK_NAME && - name === (this.info.taskFunctionNames as string[])[1]) || + // eslint-disable-next-line @typescript-eslint/no-non-null-assertion + name === this.info.taskFunctionNames![1]) || (task.name !== DEFAULT_TASK_NAME && name === task.name) ) { ++taskFunctionQueueSize