X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=src%2Fworker%2Fabstract-worker.ts;h=a84bb0691fc588d3a8e51b0fe1696d7c85bb7dc9;hb=8a11bce46f69f1183d025572345755d4fd40231b;hp=91a45e94b0f36c568326460891332d61d7de0731;hpb=241f23c2f9cfec2259c11f0a223116aca357317b;p=poolifier.git diff --git a/src/worker/abstract-worker.ts b/src/worker/abstract-worker.ts index 91a45e94..a84bb069 100644 --- a/src/worker/abstract-worker.ts +++ b/src/worker/abstract-worker.ts @@ -1,12 +1,11 @@ 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, - TaskFunctions, - WorkerAsyncFunction, - WorkerFunction, - WorkerSyncFunction + TaskPerformance, + WorkerStatistics } from '../utility-types' import { EMPTY_FUNCTION, isPlainObject } from '../utils' import { @@ -14,6 +13,12 @@ import { KillBehaviors, type WorkerOptions } from './worker-options' +import type { + TaskFunctions, + WorkerAsyncFunction, + WorkerFunction, + WorkerSyncFunction +} from './worker-functions' const DEFAULT_FUNCTION_NAME = 'default' const DEFAULT_MAX_INACTIVE_TIME = 60000 @@ -39,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. */ @@ -82,7 +91,6 @@ export abstract class AbstractWorker< ) this.checkAlive.bind(this)() } - this.mainWorker?.on('message', this.messageListener.bind(this)) } @@ -138,7 +146,9 @@ export abstract class AbstractWorker< * * @param message - Message received. */ - protected messageListener (message: MessageValue): void { + protected messageListener ( + message: MessageValue + ): void { if (message.id != null && message.data != null) { // Task message received const fn = this.getTaskFunction(message.name) @@ -154,6 +164,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 } } @@ -174,7 +187,9 @@ export abstract class AbstractWorker< * * @param message - The response message. */ - protected abstract sendToMainWorker (message: MessageValue): void + protected abstract sendToMainWorker ( + message: MessageValue + ): void /** * Checks if the worker should be terminated, because its living too long. @@ -209,21 +224,21 @@ export abstract class AbstractWorker< message: MessageValue ): void { try { - const startTimestamp = performance.now() - const waitTime = startTimestamp - (message.submissionTimestamp ?? 0) + let taskPerformance = this.beginTaskPerformance() const res = fn(message.data) - const runTime = performance.now() - startTimestamp + taskPerformance = this.endTaskPerformance(taskPerformance) this.sendToMainWorker({ data: res, - runTime, - waitTime, + taskPerformance, id: message.id }) } catch (e) { const err = this.handleError(e as Error) this.sendToMainWorker({ - error: err, - errorData: message.data, + taskError: { + message: err, + data: message.data + }, id: message.id }) } finally { @@ -241,15 +256,13 @@ export abstract class AbstractWorker< fn: WorkerAsyncFunction, message: MessageValue ): void { - const startTimestamp = performance.now() - const waitTime = startTimestamp - (message.submissionTimestamp ?? 0) + let taskPerformance = this.beginTaskPerformance() fn(message.data) .then(res => { - const runTime = performance.now() - startTimestamp + taskPerformance = this.endTaskPerformance(taskPerformance) this.sendToMainWorker({ data: res, - runTime, - waitTime, + taskPerformance, id: message.id }) return null @@ -257,8 +270,10 @@ export abstract class AbstractWorker< .catch(e => { const err = this.handleError(e as Error) this.sendToMainWorker({ - error: err, - errorData: message.data, + taskError: { + message: err, + data: message.data + }, id: message.id }) }) @@ -281,4 +296,25 @@ export abstract class AbstractWorker< } return fn } + + private beginTaskPerformance (): TaskPerformance { + return { + timestamp: performance.now(), + ...(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) + }) + } + } }