refactor: align methods signature

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

diff --git a/src/pools/pool-internal.ts b/src/pools/pool-internal.ts
index 0c2454f83f9dbebbc48358ad3ab2e7c8c81fb77d..8912dd2c18f8a99a6791b006f672442e5be5ecdf 100644 (file)
--- a/src/pools/pool-internal.ts
+++ b/src/pools/pool-internal.ts
@@ -1,48 +1,32 @@
 import type { IPool } from './pool'
-import type { IPoolWorker } from './pool-worker'
+import type { IWorker, WorkerNode } from './worker'
 
 /**
  * Internal pool types.
+ *
+ * @enum
  */
 export enum PoolType {
   FIXED = 'fixed',
   DYNAMIC = 'dynamic'
 }
 
-/**
- * Internal tasks usage statistics.
- */
-export interface TasksUsage {
-  run: number
-  running: number
-  runTime: number
-  avgRunTime: number
-}
-
 /**
  * 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 IPoolWorker,
+  Worker extends IWorker,
   Data = unknown,
   Response = unknown
 > extends IPool<Data, Response> {
   /**
-   * List of currently available workers.
+   * Pool worker nodes.
    */
-  readonly workers: Worker[]
-
-  /**
-   * The workers tasks usage map.
-   *
-   *  `key`: The `Worker`
-   *  `value`: Worker tasks usage statistics.
-   */
-  readonly workersTasksUsage: Map<Worker, TasksUsage>
+  readonly workerNodes: Array<WorkerNode<Worker, Data>>
 
   /**
    * Pool type.
@@ -52,49 +36,27 @@ export interface IPoolInternal<
   readonly type: PoolType
 
   /**
-   * Whether the pool is busy or not.
+   * Whether the pool is full or not.
    *
-   * The pool busyness boolean status.
-   */
-  readonly busy: boolean
-
-  /**
-   * Number of tasks currently concurrently running.
+   * The pool filling boolean status.
    */
-  readonly numberOfRunningTasks: number
+  readonly full: boolean
 
   /**
-   * Finds a free worker based on the number of tasks the worker has applied.
-   *
-   * If a worker is found with `0` running tasks, it is detected as free and returned.
-   *
-   * If no free worker is found, `false` is returned.
+   * Whether the pool is busy or not.
    *
-   * @returns A free worker if there is one, otherwise `false`.
+   * The pool busyness boolean status.
    */
-  findFreeWorker: () => Worker | false
+  readonly busy: boolean
 
   /**
-   * Gets worker index.
+   * Finds a free worker node key based on the number of tasks the worker has applied.
    *
-   * @param worker The worker.
-   * @returns The worker index.
-   */
-  getWorkerIndex: (worker: Worker) => number
-
-  /**
-   * Gets worker running tasks.
+   * If a worker is found with `0` running tasks, it is detected as free and its worker node key is returned.
    *
-   * @param worker The worker.
-   * @returns The number of tasks currently running on the worker.
-   */
-  getWorkerRunningTasks: (worker: Worker) => number | undefined
-
-  /**
-   * Gets worker average tasks runtime.
+   * If no free worker is found, `-1` is returned.
    *
-   * @param worker The worker.
-   * @returns The average tasks runtime on the worker.
+   * @returns A worker node key if there is one, `-1` otherwise.
    */
-  getWorkerAverageTasksRunTime: (worker: Worker) => number | undefined
+  findFreeWorkerNodeKey: () => number
 }