对事务进行排序引发错误时不进行回滚

Question

I am using sequelize transaction (managed) and when error throws rollback is not really happening, maybe i am doing something wrong here ?

my test case : when code block 2 throws an error the things done in code block 1 are not rolled back, please help.

return sequelize.transaction(function (t) {
        return db.VenueTag.destroy({where: {venueId: venue.venueId}}, {transaction: t}).then(function () {
        }, {transaction: t}).then(function () {
        //block 1
            return db.VenueTag.upsert({
            ...
            });
        }, {transaction: t}).then(function () {
        //block 2
            //update venue details
            return venue.updateAttributes({
                ...
            },{ transaction: t});
        });
    }).then(function (result) {
        // Transaction has been committed
        // result is whatever the result of the promise chain returned to the transaction callback
        return res.status(200).json(result);
    }).catch(function (err) {
        // Transaction has been rolled back
        // err is whatever rejected the promise chain returned to the transaction callback
        return res.status(500).send({
            message: errorHandler.getErrorMessage(err)
        });
    });

Answer 1

You had mistakes in your promise chain, try this:

return sequelize.transaction(function (t) {
    return db.VenueTag.destroy({where: {venueId: venue.venueId}}, {transaction: t})
    .then(function () {
    //block 1
        return db.VenueTag.upsert({
        ...
        }, {transaction: t});
    .then(function () {
    //block 2
        //update venue details
        return venue.updateAttributes({
            ...
        },{ transaction: t});
    });
}).then(function (result) {
    // Transaction has been committed
    // result is whatever the result of the promise chain returned to the transaction callback
    return res.status(200).json(result);
}).catch(function (err) {
    // Transaction has been rolled back
    // err is whatever rejected the promise chain returned to the transaction callback
    return res.status(500).send({
        message: errorHandler.getErrorMessage(err)
    });
});

Answer 2

If you're catering to an ES7 audience or using node.js. You can use the async/await approach to simplify the callback/promise chain:

    /*Global Setup*/
    let venue, db; // Whatever instances these are....

    async function deleteVenueTag () {
      let transaction;
      try {
        transaction = await sequelize.transaction();
        let VenueTag = await db.findOne({where: {venueId: venue.venueId}}, {transaction});
        if (VenueTag) {
          await VenueTag.destroy({ transaction });
          await VenueTag.upsert({ transaction });
          await venue.updateAttributes({ transaction });
          await transaction.commit();
        }
      } catch (error) {
        transaction.rollback();
      }
    }