refactor: untangle worker choosing code from worker creation code

[poolifier.git] / src / pools / selection-strategies / selection-strategies-types.ts

diff --git a/src/pools/selection-strategies/selection-strategies-types.ts b/src/pools/selection-strategies/selection-strategies-types.ts
index f875c2906ee3a173ffab6790d93aece511b2a0ef..9acf819b8b858900a1d42a5a681d7f001b152edf 100644 (file)
--- a/src/pools/selection-strategies/selection-strategies-types.ts
+++ b/src/pools/selection-strategies/selection-strategies-types.ts
@@ -1,5 +1,3 @@
-import type { IPoolWorker } from '../pool-worker'
-
 /**
  * Enumeration of worker choice strategies.
  */
@@ -9,9 +7,13 @@ export const WorkerChoiceStrategies = Object.freeze({
    */
   ROUND_ROBIN: 'ROUND_ROBIN',
   /**
-   * Less recently used worker selection strategy.
+   * Less used worker selection strategy.
+   */
+  LESS_USED: 'LESS_USED',
+  /**
+   * Less busy worker selection strategy.
    */
-  LESS_RECENTLY_USED: 'LESS_RECENTLY_USED',
+  LESS_BUSY: 'LESS_BUSY',
   /**
    * Fair share worker selection strategy.
    */
@@ -28,20 +30,19 @@ export const WorkerChoiceStrategies = Object.freeze({
 export type WorkerChoiceStrategy = keyof typeof WorkerChoiceStrategies
 
 /**
- * Pool tasks usage statistics requirements.
+ * Pool worker tasks usage statistics requirements.
  */
-export type RequiredStatistics = {
+export interface RequiredStatistics {
   runTime: boolean
+  avgRunTime: boolean
 }
 
 /**
  * Worker choice strategy interface.
- *
- * @template Worker Type of worker which manages the strategy.
  */
-export interface IWorkerChoiceStrategy<Worker extends IPoolWorker> {
+export interface IWorkerChoiceStrategy {
   /**
-   * Is the pool attached to the strategy dynamic?.
+   * Is the pool bound to the strategy dynamic?.
    */
   readonly isDynamicPool: boolean
   /**
@@ -49,11 +50,17 @@ export interface IWorkerChoiceStrategy<Worker extends IPoolWorker> {
    */
   readonly requiredStatistics: RequiredStatistics
   /**
-   * Resets strategy internal statistics.
+   * Resets strategy internals (counters, statistics, etc.).
+   */
+  reset: () => boolean
+  /**
+   * Chooses a worker in the pool and returns its key.
    */
-  resetStatistics(): boolean
+  choose: () => number
   /**
-   * Chooses a worker in the pool.
+   * Removes a worker reference from strategy internals.
+   *
+   * @param workerKey - The worker key.
    */
-  choose(): Worker
+  remove: (workerKey: number) => boolean
 }