typeorm 0.2.44-dev.8742e33 → 0.2.45-dev.0fc093d

package/browser/driver/cockroachdb/CockroachQueryRunner.js

@@ -1,6 +1,5 @@
 import { __awaiter, __extends, __generator, __read, __spreadArray, __values } from "tslib";
 import { QueryResult } from "../../query-runner/QueryResult";
-import { TransactionAlreadyStartedError } from "../../error/TransactionAlreadyStartedError";
 import { TransactionNotStartedError } from "../../error/TransactionNotStartedError";
 import { TableColumn } from "../../schema-builder/table/TableColumn";
 import { Table } from "../../schema-builder/table/Table";
@@ -93,30 +92,45 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
      */
     CockroachQueryRunner.prototype.startTransaction = function (isolationLevel) {
         return __awaiter(this, void 0, void 0, function () {
+            var err_1;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
-                        if (this.isTransactionActive)
-                            throw new TransactionAlreadyStartedError();
-                        return [4 /*yield*/, this.broadcaster.broadcast('BeforeTransactionStart')];
+                        this.isTransactionActive = true;
+                        _a.label = 1;
                     case 1:
+                        _a.trys.push([1, 3, , 4]);
+                        return [4 /*yield*/, this.broadcaster.broadcast('BeforeTransactionStart')];
+                    case 2:
                         _a.sent();
-                        this.isTransactionActive = true;
+                        return [3 /*break*/, 4];
+                    case 3:
+                        err_1 = _a.sent();
+                        this.isTransactionActive = false;
+                        throw err_1;
+                    case 4:
+                        if (!(this.transactionDepth === 0)) return [3 /*break*/, 9];
                         return [4 /*yield*/, this.query("START TRANSACTION")];
-                    case 2:
+                    case 5:
                         _a.sent();
                         return [4 /*yield*/, this.query("SAVEPOINT cockroach_restart")];
-                    case 3:
+                    case 6:
                         _a.sent();
-                        if (!isolationLevel) return [3 /*break*/, 5];
+                        if (!isolationLevel) return [3 /*break*/, 8];
                         return [4 /*yield*/, this.query("SET TRANSACTION ISOLATION LEVEL " + isolationLevel)];
-                    case 4:
+                    case 7:
                         _a.sent();
-                        _a.label = 5;
-                    case 5:
+                        _a.label = 8;
+                    case 8: return [3 /*break*/, 11];
+                    case 9: return [4 /*yield*/, this.query("SAVEPOINT typeorm_" + this.transactionDepth)];
+                    case 10:
+                        _a.sent();
+                        _a.label = 11;
+                    case 11:
+                        this.transactionDepth += 1;
                         this.storeQueries = true;
                         return [4 /*yield*/, this.broadcaster.broadcast('AfterTransactionStart')];
-                    case 6:
+                    case 12:
                         _a.sent();
                         return [2 /*return*/];
                 }
@@ -139,58 +153,66 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
                         return [4 /*yield*/, this.broadcaster.broadcast('BeforeTransactionCommit')];
                     case 1:
                         _d.sent();
-                        this.storeQueries = false;
-                        _d.label = 2;
+                        if (!(this.transactionDepth > 1)) return [3 /*break*/, 3];
+                        return [4 /*yield*/, this.query("RELEASE SAVEPOINT typeorm_" + (this.transactionDepth - 1))];
                     case 2:
-                        _d.trys.push([2, 5, , 17]);
-                        return [4 /*yield*/, this.query("RELEASE SAVEPOINT cockroach_restart")];
+                        _d.sent();
+                        this.transactionDepth -= 1;
+                        return [3 /*break*/, 19];
                     case 3:
+                        this.storeQueries = false;
+                        _d.label = 4;
+                    case 4:
+                        _d.trys.push([4, 7, , 19]);
+                        return [4 /*yield*/, this.query("RELEASE SAVEPOINT cockroach_restart")];
+                    case 5:
                         _d.sent();
                         return [4 /*yield*/, this.query("COMMIT")];
-                    case 4:
+                    case 6:
                         _d.sent();
                         this.queries = [];
                         this.isTransactionActive = false;
-                        return [3 /*break*/, 17];
-                    case 5:
+                        this.transactionDepth -= 1;
+                        return [3 /*break*/, 19];
+                    case 7:
                         e_1 = _d.sent();
-                        if (!(e_1.code === "40001")) return [3 /*break*/, 16];
+                        if (!(e_1.code === "40001")) return [3 /*break*/, 18];
                         return [4 /*yield*/, this.query("ROLLBACK TO SAVEPOINT cockroach_restart")];
-                    case 6:
+                    case 8:
                         _d.sent();
-                        _d.label = 7;
-                    case 7:
-                        _d.trys.push([7, 12, 13, 14]);
+                        _d.label = 9;
+                    case 9:
+                        _d.trys.push([9, 14, 15, 16]);
                         _a = __values(this.queries), _b = _a.next();
-                        _d.label = 8;
-                    case 8:
-                        if (!!_b.done) return [3 /*break*/, 11];
+                        _d.label = 10;
+                    case 10:
+                        if (!!_b.done) return [3 /*break*/, 13];
                         q = _b.value;
                         return [4 /*yield*/, this.query(q.query, q.parameters)];
-                    case 9:
+                    case 11:
                         _d.sent();
-                        _d.label = 10;
-                    case 10:
-                        _b = _a.next();
-                        return [3 /*break*/, 8];
-                    case 11: return [3 /*break*/, 14];
+                        _d.label = 12;
                     case 12:
+                        _b = _a.next();
+                        return [3 /*break*/, 10];
+                    case 13: return [3 /*break*/, 16];
+                    case 14:
                         e_2_1 = _d.sent();
                         e_2 = { error: e_2_1 };
-                        return [3 /*break*/, 14];
-                    case 13:
+                        return [3 /*break*/, 16];
+                    case 15:
                         try {
                             if (_b && !_b.done && (_c = _a.return)) _c.call(_a);
                         }
                         finally { if (e_2) throw e_2.error; }
                         return [7 /*endfinally*/];
-                    case 14: return [4 /*yield*/, this.commitTransaction()];
-                    case 15:
+                    case 16: return [4 /*yield*/, this.commitTransaction()];
+                    case 17:
                         _d.sent();
-                        _d.label = 16;
-                    case 16: return [3 /*break*/, 17];
-                    case 17: return [4 /*yield*/, this.broadcaster.broadcast('AfterTransactionCommit')];
-                    case 18:
+                        _d.label = 18;
+                    case 18: return [3 /*break*/, 19];
+                    case 19: return [4 /*yield*/, this.broadcaster.broadcast('AfterTransactionCommit')];
+                    case 20:
                         _d.sent();
                         return [2 /*return*/];
                 }
@@ -211,14 +233,23 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
                         return [4 /*yield*/, this.broadcaster.broadcast('BeforeTransactionRollback')];
                     case 1:
                         _a.sent();
+                        if (!(this.transactionDepth > 1)) return [3 /*break*/, 3];
+                        return [4 /*yield*/, this.query("ROLLBACK TO SAVEPOINT typeorm_" + (this.transactionDepth - 1))];
+                    case 2:
+                        _a.sent();
+                        return [3 /*break*/, 5];
+                    case 3:
                         this.storeQueries = false;
                         return [4 /*yield*/, this.query("ROLLBACK")];
-                    case 2:
+                    case 4:
                         _a.sent();
                         this.queries = [];
                         this.isTransactionActive = false;
+                        _a.label = 5;
+                    case 5:
+                        this.transactionDepth -= 1;
                         return [4 /*yield*/, this.broadcaster.broadcast('AfterTransactionRollback')];
-                    case 3:
+                    case 6:
                         _a.sent();
                         return [2 /*return*/];
                 }
@@ -231,7 +262,7 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
     CockroachQueryRunner.prototype.query = function (query, parameters, useStructuredResult) {
         if (useStructuredResult === void 0) { useStructuredResult = false; }
         return __awaiter(this, void 0, void 0, function () {
-            var databaseConnection, queryStartTime, raw, maxQueryExecutionTime, queryEndTime, queryExecutionTime, result, err_1;
+            var databaseConnection, queryStartTime, raw, maxQueryExecutionTime, queryEndTime, queryExecutionTime, result, err_2;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
@@ -282,11 +313,11 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
                         }
                         return [3 /*break*/, 5];
                     case 4:
-                        err_1 = _a.sent();
-                        if (err_1.code !== "40001") {
-                            this.driver.connection.logger.logQueryError(err_1, query, parameters, this);
+                        err_2 = _a.sent();
+                        if (err_2.code !== "40001") {
+                            this.driver.connection.logger.logQueryError(err_2, query, parameters, this);
                         }
-                        throw new QueryFailedError(query, parameters, err_1);
+                        throw new QueryFailedError(query, parameters, err_2);
                     case 5: return [2 /*return*/];
                 }
             });
@@ -2016,7 +2047,7 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
      */
     CockroachQueryRunner.prototype.clearDatabase = function () {
         return __awaiter(this, void 0, void 0, function () {
-            var schemas, schemaNamesString, selectViewDropsQuery, dropViewQueries, selectDropsQuery, dropQueries, selectSequenceDropsQuery, sequenceDropQueries, error_1, rollbackError_1;
+            var schemas, schemaNamesString, isAnotherTransactionActive, selectViewDropsQuery, dropViewQueries, selectDropsQuery, dropQueries, selectSequenceDropsQuery, sequenceDropQueries, error_1, rollbackError_1;
             var _this = this;
             return __generator(this, function (_a) {
                 switch (_a.label) {
@@ -2033,12 +2064,14 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
                         schemaNamesString = schemas.map(function (name) {
                             return name === "current_schema()" ? name : "'" + name + "'";
                         }).join(", ");
+                        isAnotherTransactionActive = this.isTransactionActive;
+                        if (!!isAnotherTransactionActive) return [3 /*break*/, 2];
                         return [4 /*yield*/, this.startTransaction()];
                     case 1:
                         _a.sent();
                         _a.label = 2;
                     case 2:
-                        _a.trys.push([2, 10, , 15]);
+                        _a.trys.push([2, 11, , 17]);
                         selectViewDropsQuery = "SELECT 'DROP VIEW IF EXISTS \"' || schemaname || '\".\"' || viewname || '\" CASCADE;' as \"query\" " +
                             ("FROM \"pg_views\" WHERE \"schemaname\" IN (" + schemaNamesString + ")");
                         return [4 /*yield*/, this.query(selectViewDropsQuery)];
@@ -2061,24 +2094,28 @@ var CockroachQueryRunner = /** @class */ (function (_super) {
                         return [4 /*yield*/, Promise.all(sequenceDropQueries.map(function (q) { return _this.query(q["query"]); }))];
                     case 8:
                         _a.sent();
+                        if (!!isAnotherTransactionActive) return [3 /*break*/, 10];
                         return [4 /*yield*/, this.commitTransaction()];
                     case 9:
                         _a.sent();
-                        return [3 /*break*/, 15];
-                    case 10:
-                        error_1 = _a.sent();
-                        _a.label = 11;
+                        _a.label = 10;
+                    case 10: return [3 /*break*/, 17];
                     case 11:
-                        _a.trys.push([11, 13, , 14]);
-                        return [4 /*yield*/, this.rollbackTransaction()];
+                        error_1 = _a.sent();
+                        _a.label = 12;
                     case 12:
-                        _a.sent();
-                        return [3 /*break*/, 14];
+                        _a.trys.push([12, 15, , 16]);
+                        if (!!isAnotherTransactionActive) return [3 /*break*/, 14];
+                        return [4 /*yield*/, this.rollbackTransaction()];
                     case 13:
+                        _a.sent();
+                        _a.label = 14;
+                    case 14: return [3 /*break*/, 16];
+                    case 15:
                         rollbackError_1 = _a.sent();
-                        return [3 /*break*/, 14];
-                    case 14: throw error_1;
-                    case 15: return [2 /*return*/];
+                        return [3 /*break*/, 16];
+                    case 16: throw error_1;
+                    case 17: return [2 /*return*/];
                 }
             });
         });