@naturalcycles/abba 1.25.3 → 1.26.0

package/dist/abba.d.ts

@@ -36,6 +36,8 @@ export declare class Abba {
     private updateExclusions;
     /**
      * Delete an experiment. Removes all user assignments and buckets.
+     * Requires the experiment to have been inactive for at least 15 minutes in order to
+     * avoid row locking issues.
      * Cold method.
      */
     deleteExperiment(experimentId: string): Promise<void>;

package/dist/abba.js

@@ -12,7 +12,7 @@ const util_1 = require("./util");
 /**
  * 10 minutes
  */
-const CACHE_TTL = 600_000; // 10 minutes
+const CACHE_TTL = 600_000;
 class Abba {
     constructor(cfg) {
         this.cfg = cfg;
@@ -103,9 +103,15 @@ class Abba {
     }
     /**
      * Delete an experiment. Removes all user assignments and buckets.
+     * Requires the experiment to have been inactive for at least 15 minutes in order to
+     * avoid row locking issues.
      * Cold method.
      */
     async deleteExperiment(experimentId) {
+        const experiment = await this.experimentDao.requireById(experimentId);
+        const hasBeenInactiveFor15Mins = experiment.status === types_1.AssignmentStatus.Inactive &&
+            (0, js_lib_1.localTime)(experiment.updated).isOlderThan(15, 'minute');
+        (0, js_lib_1._assert)(hasBeenInactiveFor15Mins, 'Experiment must be inactive for at least 15 minutes before deletion');
         const userAssignmentDeleteQuery = this.userAssignmentDao
             .query()
             .filterEq('experimentId', experimentId);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@naturalcycles/abba",
-  "version": "1.25.3",
+  "version": "1.26.0",
   "scripts": {
     "prepare": "husky",
     "build": "dev-lib build",

package/src/abba.ts

@@ -1,4 +1,4 @@
-import { _assert, _Memo, _shuffle, pMap, Unsaved } from '@naturalcycles/js-lib'
+import { _assert, _Memo, _shuffle, localTime, pMap, Unsaved } from '@naturalcycles/js-lib'
 import { LRUMemoCache } from '@naturalcycles/nodejs-lib'
 import { SegmentationData } from '.'
 import { bucketDao } from './dao/bucket.dao'
@@ -25,7 +25,7 @@ import {
 /**
  * 10 minutes
  */
-const CACHE_TTL = 600_000 // 10 minutes
+const CACHE_TTL = 600_000
 
 export class Abba {
   private experimentDao = experimentDao(this.cfg.db)
@@ -149,9 +149,21 @@ export class Abba {
 
   /**
    * Delete an experiment. Removes all user assignments and buckets.
+   * Requires the experiment to have been inactive for at least 15 minutes in order to
+   * avoid row locking issues.
    * Cold method.
    */
   async deleteExperiment(experimentId: string): Promise<void> {
+    const experiment = await this.experimentDao.requireById(experimentId)
+
+    const hasBeenInactiveFor15Mins =
+      experiment.status === AssignmentStatus.Inactive &&
+      localTime(experiment.updated).isOlderThan(15, 'minute')
+    _assert(
+      hasBeenInactiveFor15Mins,
+      'Experiment must be inactive for at least 15 minutes before deletion',
+    )
+
     const userAssignmentDeleteQuery = this.userAssignmentDao
       .query()
       .filterEq('experimentId', experimentId)