@cubejs-backend/cubestore - npm Package Compare versions

Comparing version 0.26.86 to 0.26.87

CHANGELOG.md

@@ -6,2 +6,13 @@ # Change Log
 
+## [0.26.87](https://github.com/cube-js/cube.js/compare/v0.26.86...v0.26.87) (2021-04-10)
+
+
+### Bug Fixes
+
+* **cubestore:** Something wrong with downloading Cube Store before running it. ([208dd31](https://github.com/cube-js/cube.js/commit/208dd31f20aa64a5c79e143d40055ac2658d0745))
+
+
+
+
+
 ## [0.26.86](https://github.com/cube-js/cube.js/compare/v0.26.85...v0.26.86) (2021-04-09)
@@ -8,0 +19,0 @@

dist/process.d.ts

@@ -12,7 +12,8 @@ /// <reference types="node" />
     protected cubeStoreStarting: Promise<ChildProcess> | null;
+    protected releaseRequested: boolean;
     constructor(config: Readonly<CubeStoreHandlerOptions>);
     protected getBinary(): Promise<string>;
     acquire(): Promise<ChildProcess>;
-    release(): Promise<void>;
+    release(force?: boolean): Promise<void>;
 }
 //# sourceMappingURL=process.d.ts.map

dist/process.js

@@ -84,2 +84,4 @@ "use strict";
         this.cubeStoreStarting = null;
+        // Flag when release was requested, in this state, we skip restart on exit
+        this.releaseRequested = false;
     }
@@ -104,2 +106,5 @@ async getBinary() {
         const onExit = (code) => {
+            if (this.releaseRequested) {
+                return;
+            }
             this.config.onRestart(code);
@@ -141,4 +146,13 @@ this.cubeStoreStarting = new Promise((resolve, reject) => startProcess(download_1.getBinaryPath(), {
     }
-    async release() {
-        // @todo Use SIGTERM for gracefully shutdown?
+    async release(force = false) {
+        // Force, is a compatibility flag, for now we release only in tests
+        if (force) {
+            if (this.cubeStoreStarting) {
+                throw new Error('Something wrong with logic, release was called, while cubestore is starting...');
+            }
+            this.releaseRequested = true;
+            if (this.cubeStore) {
+                this.cubeStore.kill('SIGTERM');
+            }
+        }
     }
@@ -145,0 +159,0 @@ }

package.json

 {
   "name": "@cubejs-backend/cubestore",
-  "version": "0.26.86",
+  "version": "0.26.87",
   "description": "Cube.js pre-aggregation storage layer.",
@@ -19,2 +19,4 @@ "main": "dist/index.js",
     "lint:fix": "eslint --fix js-wrapper/* --ext .ts,js",
+    "unit": "jest",
+    "unit:debug": "jest --runInBand",
     "postinstall": "bin/post-install"
@@ -30,3 +32,5 @@ },
     "@cubejs-backend/linter": "^0.26.74",
+    "@types/jest": "^26.0.22",
     "@types/node": "^10.17.54",
+    "jest": "^26.6.3",
     "typescript": "~4.1.5"
@@ -38,3 +42,3 @@ },
   "dependencies": {
-    "@cubejs-backend/shared": "^0.26.81",
+    "@cubejs-backend/shared": "^0.26.87",
     "@octokit/core": "^3.2.5",
@@ -46,3 +50,9 @@ "source-map-support": "^0.5.19"
   },
-  "gitHead": "69ea7cdadd28f7d3bf1fb4b1c33506dd3e1cd36a"
+  "jest": {
+    "testMatch": [
+      "<rootDir>/dist/*.(test|spec).(js)"
+    ],
+    "testEnvironment": "node"
+  },
+  "gitHead": "59c0bd83e2651f0bf058287d7d75a6a72fb6dd7d"
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

365636

increased by1.24%

Number of package files

35

increased by12.9%

Lines of code

465

increased by12.32%

Worsened metrics

Dev dependency count

5

increased by66.67%

Number of low supply chain risk alerts

4

increased by33.33%

Dependency changes

• Updated@cubejs-backend/shared@^0.26.87