import { fork, isMaster, setupMaster, Worker } from 'cluster'
-import type { JSONValue, MessageValue } from '../../utility-types'
+import type { MessageValue } from '../../utility-types'
import type { PoolOptions } from '../abstract-pool'
import { AbstractPool } from '../abstract-pool'
+import { PoolType } from '../pool-internal'
/**
* Options for a poolifier cluster pool.
*
* This pool selects the workers in a round robin fashion.
*
- * @template Data Type of data sent to the worker.
- * @template Response Type of response of execution.
+ * @template Data Type of data sent to the worker. This can only be serializable data.
+ * @template Response Type of response of execution. This can only be serializable data.
*
* @author [Christopher Quadflieg](https://github.com/Shinigami92)
* @since 2.0.0
*/
export class FixedClusterPool<
- Data extends JSONValue = JSONValue,
- Response extends JSONValue = JSONValue
+ Data = unknown,
+ Response = unknown
> extends AbstractPool<Worker, Data, Response> {
/**
* Constructs a new poolifier fixed cluster pool.
*
- * @param numWorkers Number of workers for this pool.
+ * @param numberOfWorkers Number of workers for this pool.
* @param filePath Path to an implementation of a `ClusterWorker` file, which can be relative or absolute.
- * @param opts Options for this fixed cluster pool. Default: `{ maxTasks: 1000 }`
+ * @param opts Options for this fixed cluster pool. Default: `{}`
*/
public constructor (
- numWorkers: number,
+ numberOfWorkers: number,
filePath: string,
- public readonly opts: ClusterPoolOptions = { maxTasks: 1000 }
+ public readonly opts: ClusterPoolOptions = {}
) {
- super(numWorkers, filePath, opts)
+ super(numberOfWorkers, filePath, opts)
}
+ /** @inheritdoc */
protected setupHook (): void {
setupMaster({
exec: this.filePath
})
}
+ /** @inheritdoc */
protected isMain (): boolean {
return isMaster
}
- protected destroyWorker (worker: Worker): void {
+ /** @inheritdoc */
+ public destroyWorker (worker: Worker): void {
+ this.sendToWorker(worker, { kill: 1 })
worker.kill()
- // FIXME: The tests are currently failing, so these must be changed first
}
+ /** @inheritdoc */
protected sendToWorker (worker: Worker, message: MessageValue<Data>): void {
worker.send(message)
}
- protected registerWorkerMessageListener (
- port: Worker,
- listener: (message: MessageValue<Response>) => void
+ /** @inheritdoc */
+ public registerWorkerMessageListener<Message extends Data | Response> (
+ worker: Worker,
+ listener: (message: MessageValue<Message>) => void
): void {
- port.on('message', listener)
+ worker.on('message', listener)
}
- protected unregisterWorkerMessageListener (
- port: Worker,
- listener: (message: MessageValue<Response>) => void
- ): void {
- port.removeListener('message', listener)
+ /** @inheritdoc */
+ protected createWorker (): Worker {
+ return fork(this.opts.env)
}
- protected newWorker (): Worker {
- return fork(this.opts.env)
+ /** @inheritdoc */
+ protected afterWorkerSetup (worker: Worker): void {
+ // Listen worker messages.
+ this.registerWorkerMessageListener(worker, super.workerListener())
+ }
+
+ /** @inheritdoc */
+ public get type (): PoolType {
+ return PoolType.FIXED
}
- protected afterNewWorkerPushed (worker: Worker): void {
- // we will attach a listener for every task,
- // when task is completed the listener will be removed but to avoid warnings we are increasing the max listeners size
- worker.setMaxListeners(this.opts.maxTasks ?? 1000)
+ /** @inheritdoc */
+ public get busy (): boolean {
+ return this.internalGetBusyStatus()
}
}