refactor: remove unneeded encapsulation around tasks usage handling

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

diff --git a/src/pools/pool-internal.ts b/src/pools/pool-internal.ts
index 470ccdd5938e04ac8585ddc1fb6afc906919d02e..9da6495266ab6c6e4654def94eca460b4630cb24 100644 (file)
--- a/src/pools/pool-internal.ts
+++ b/src/pools/pool-internal.ts
@@ -19,12 +19,22 @@ export interface TasksUsage {
   avgRunTime: number
 }
 
+/**
+ * Internal worker type.
+ *
+ * @typeParam Worker - Type of worker 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.
+ * @typeParam Response - Type of response of execution.
  */
 export interface IPoolInternal<
   Worker extends IPoolWorker,
@@ -32,17 +42,9 @@ export interface IPoolInternal<
   Response = unknown
 > extends IPool<Data, Response> {
   /**
-   * List of currently available workers.
+   * Pool workers map.
    */
-  readonly workers: Worker[]
-
-  /**
-   * The workers tasks usage map.
-   *
-   *  `key`: The `Worker`
-   *  `value`: Worker tasks usage statistics.
-   */
-  readonly workersTasksUsage: Map<Worker, TasksUsage>
+  readonly workers: Map<number, WorkerType<Worker>>
 
   /**
    * Pool type.
@@ -51,11 +53,6 @@ export interface IPoolInternal<
    */
   readonly type: PoolType
 
-  /**
-   * Maximum number of workers that can be created by this pool.
-   */
-  readonly max?: number
-
   /**
    * Whether the pool is busy or not.
    *
@@ -77,29 +74,13 @@ export interface IPoolInternal<
    *
    * @returns A free worker if there is one, otherwise `false`.
    */
-  findFreeWorker(): Worker | false
-
-  /**
-   * Gets worker index.
-   *
-   * @param worker The worker.
-   * @returns The worker index.
-   */
-  getWorkerIndex(worker: Worker): number
-
-  /**
-   * Gets worker running tasks.
-   *
-   * @param worker The worker.
-   * @returns The number of tasks currently running on the worker.
-   */
-  getWorkerRunningTasks(worker: Worker): number | undefined
+  findFreeWorker: () => Worker | false
 
   /**
-   * Gets worker average tasks runtime.
+   * Gets worker tasks usage.
    *
-   * @param worker The worker.
-   * @returns The average tasks runtime on the worker.
+   * @param worker - The worker.
+   * @returns The tasks usage on the worker.
    */
-  getWorkerAverageTasksRunTime(worker: Worker): number | undefined
+  getWorkerTasksUsage: (worker: Worker) => TasksUsage | undefined
 }