X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Fworker%2Fabstract-worker.ts;h=7805514a7b3b083385b96ac4d26218840b625b31;hb=d715b7bc2973eed82edebb1f7d233d451ad3c97b;hp=163736696b8ec7080ee17556a52e18aa40e07466;hpb=f34fdabe7562f0c1da21643162d5b6c8ad298812;p=poolifier.git diff --git a/src/worker/abstract-worker.ts b/src/worker/abstract-worker.ts index 16373669..7805514a 100644 --- a/src/worker/abstract-worker.ts +++ b/src/worker/abstract-worker.ts @@ -1,16 +1,24 @@ import { AsyncResource } from 'node:async_hooks' import type { Worker } from 'node:cluster' import type { MessagePort } from 'node:worker_threads' +import { performance } from 'node:perf_hooks' import type { MessageValue, + TaskPerformance, + WorkerStatistics +} from '../utility-types' +import { EMPTY_FUNCTION, isPlainObject } from '../utils' +import { + type KillBehavior, + KillBehaviors, + type WorkerOptions +} from './worker-options' +import type { TaskFunctions, WorkerAsyncFunction, WorkerFunction, WorkerSyncFunction -} from '../utility-types' -import { EMPTY_FUNCTION, isPlainObject } from '../utils' -import type { KillBehavior, WorkerOptions } from './worker-options' -import { KillBehaviors } from './worker-options' +} from './worker-functions' const DEFAULT_FUNCTION_NAME = 'default' const DEFAULT_MAX_INACTIVE_TIME = 60000 @@ -36,6 +44,10 @@ export abstract class AbstractWorker< * Timestamp of the last task processed by this worker. */ protected lastTaskTimestamp!: number + /** + * Performance statistics computation. + */ + protected statistics!: WorkerStatistics /** * Handler id of the `aliveInterval` worker alive check. */ @@ -79,7 +91,6 @@ export abstract class AbstractWorker< ) this.checkAlive.bind(this)() } - this.mainWorker?.on('message', this.messageListener.bind(this)) } @@ -87,7 +98,7 @@ export abstract class AbstractWorker< this.opts.killBehavior = opts.killBehavior ?? DEFAULT_KILL_BEHAVIOR this.opts.maxInactiveTime = opts.maxInactiveTime ?? DEFAULT_MAX_INACTIVE_TIME - this.opts.async = opts.async ?? false + delete this.opts.async } /** @@ -151,6 +162,9 @@ export abstract class AbstractWorker< // Kill message received this.aliveInterval != null && clearInterval(this.aliveInterval) this.emitDestroy() + } else if (message.statistics != null) { + // Statistics message received + this.statistics = message.statistics } } @@ -206,17 +220,21 @@ export abstract class AbstractWorker< message: MessageValue ): void { try { - const startTimestamp = performance.now() + let taskPerformance = this.beginTaskPerformance(message) const res = fn(message.data) - const runTime = performance.now() - startTimestamp + taskPerformance = this.endTaskPerformance(taskPerformance) this.sendToMainWorker({ data: res, - id: message.id, - runTime + taskPerformance, + id: message.id }) } catch (e) { const err = this.handleError(e as Error) - this.sendToMainWorker({ error: err, id: message.id }) + this.sendToMainWorker({ + error: err, + errorData: message.data, + id: message.id + }) } finally { !this.isMain && (this.lastTaskTimestamp = performance.now()) } @@ -232,20 +250,24 @@ export abstract class AbstractWorker< fn: WorkerAsyncFunction, message: MessageValue ): void { - const startTimestamp = performance.now() + let taskPerformance = this.beginTaskPerformance(message) fn(message.data) .then(res => { - const runTime = performance.now() - startTimestamp + taskPerformance = this.endTaskPerformance(taskPerformance) this.sendToMainWorker({ data: res, - id: message.id, - runTime + taskPerformance, + id: message.id }) return null }) .catch(e => { const err = this.handleError(e as Error) - this.sendToMainWorker({ error: err, id: message.id }) + this.sendToMainWorker({ + error: err, + errorData: message.data, + id: message.id + }) }) .finally(() => { !this.isMain && (this.lastTaskTimestamp = performance.now()) @@ -262,8 +284,33 @@ export abstract class AbstractWorker< name = name ?? DEFAULT_FUNCTION_NAME const fn = this.taskFunctions.get(name) if (fn == null) { - throw new Error(`Task function "${name}" not found`) + throw new Error(`Task function '${name}' not found`) } return fn } + + private beginTaskPerformance (message: MessageValue): TaskPerformance { + const timestamp = performance.now() + return { + timestamp, + ...(this.statistics.waitTime && { + waitTime: timestamp - (message.timestamp ?? timestamp) + }), + ...(this.statistics.elu && { elu: performance.eventLoopUtilization() }) + } + } + + private endTaskPerformance ( + taskPerformance: TaskPerformance + ): TaskPerformance { + return { + ...taskPerformance, + ...(this.statistics.runTime && { + runTime: performance.now() - taskPerformance.timestamp + }), + ...(this.statistics.elu && { + elu: performance.eventLoopUtilization(taskPerformance.elu) + }) + } + } }