perf: remove unneeded nullish checks in hot code paths

[poolifier.git] / src / pools / pool-internal.ts

diff --git a/src/pools/pool-internal.ts b/src/pools/pool-internal.ts
index dd71ae71045b555d99b64a24bba516461c77b561..095e954e2c2bb498ad27f1e1a5999686b99249a7 100644 (file)
--- a/src/pools/pool-internal.ts
+++ b/src/pools/pool-internal.ts
@@ -1,9 +1,8 @@
-import EventEmitter from 'events'
-import type { IWorker } from './abstract-pool'
 import type { IPool } from './pool'
+import type { IPoolWorker } from './pool-worker'
 
 /**
- * Pool types.
+ * Internal pool types.
  */
 export enum PoolType {
   FIXED = 'fixed',
@@ -11,43 +10,42 @@ export enum PoolType {
 }
 
 /**
- * Internal poolifier pool emitter.
+ * Internal tasks usage statistics.
  */
-export class PoolEmitter extends EventEmitter {}
+export interface TasksUsage {
+  run: number
+  running: number
+  runTime: number
+  avgRunTime: number
+  error: number
+}
+
+/**
+ * Internal worker type.
+ *
+ * @typeParam Worker - Type of worker type items which manages this pool.
+ */
+export interface WorkerType<Worker extends IPoolWorker> {
+  worker: Worker
+  tasksUsage: TasksUsage
+}
 
 /**
  * Internal contract definition for a poolifier pool.
  *
- * @template Worker Type of worker which manages this pool.
- * @template Data Type of data sent to the worker.
- * @template Response Type of response of execution.
+ * @typeParam Worker - Type of worker which manages this pool.
+ * @typeParam Data - Type of data sent to the worker. This can only be serializable data.
+ * @typeParam Response - Type of response of execution. This can only be serializable data.
  */
 export interface IPoolInternal<
-  Worker extends IWorker,
+  Worker extends IPoolWorker,
   Data = unknown,
   Response = unknown
 > extends IPool<Data, Response> {
   /**
-   * List of currently available workers.
-   */
-  readonly workers: Worker[]
-
-  /**
-   * The tasks map.
-   *
-   * - `key`: The `Worker`
-   * - `value`: Number of tasks currently in progress on the worker.
+   * Pool worker type items array.
    */
-  readonly tasks: Map<Worker, number>
-
-  /**
-   * Emitter on which events can be listened to.
-   *
-   * Events that can currently be listened to:
-   *
-   * - `'busy'`
-   */
-  readonly emitter?: PoolEmitter
+  readonly workers: Array<WorkerType<Worker>>
 
   /**
    * Pool type.
@@ -57,9 +55,11 @@ export interface IPoolInternal<
   readonly type: PoolType
 
   /**
-   * Maximum number of workers that can be created by this pool.
+   * Whether the pool is full or not.
+   *
+   * The pool filling boolean status.
    */
-  readonly max?: number
+  readonly full: boolean
 
   /**
    * Whether the pool is busy or not.
@@ -69,18 +69,13 @@ export interface IPoolInternal<
   readonly busy: boolean
 
   /**
-   * Number of tasks currently concurrently running.
-   */
-  readonly numberOfRunningTasks: number
-
-  /**
-   * Find a tasks map entry with a free worker based on the number of tasks the worker has applied.
+   * Finds a free worker key based on the number of tasks the worker has applied.
    *
-   * If an entry is found with a worker that has `0` tasks, it is detected as free.
+   * If a worker is found with `0` running tasks, it is detected as free and its key is returned.
    *
-   * If no tasks map entry with a free worker was found, `false` will be returned.
+   * If no free worker is found, `false` is returned.
    *
-   * @returns A tasks map entry with a free worker if there was one, otherwise `false`.
+   * @returns A worker key if there is one, `-1` otherwise.
    */
-  findFreeTasksMapEntry(): [Worker, number] | false
+  findFreeWorkerKey: () => number
 }