在唯一键冲突后,node-mysql事务不会回滚

时间:2017-03-08 18:51:11

标签: node.js transactions node-mysql

我首先执行UPDATE后,通过执行违反UNIQUE键约束的INSERT来测试我的应用程序中的node-mysql事务。虽然INSERT失败,但我发现初始UPDATE正在成功,尽管与ROLLBACK进行了交易。 MySQL UPDATE没有执行隐式提交,因此我不确定这里到底发生了什么。

pool.getConnection(function (err, connection) {

    connection.beginTransaction(function (err) {
        if (err) {
            console.log(" --- error 1: " + JSON.stringify(err));
        }

        console.log(" --- A");

        connection.query('update course_person set rescheduled_date = CURDATE() where idperson = ? and rescheduled_date is null', [idperson], function (err, rows, fields) {

            if (err) {
                connection.rollback(function () {
                    console.log(" --- error 2: " + JSON.stringify(err));
                });

                console.log(" --- B");

            }

            console.log(" --- C");

        });

        connection.query('insert into course_person (idcourse, idperson) VALUES (?, ?)', [newidcourse, idperson], function (err, rows, fields) {

            if (err) {
                connection.rollback(function () {
                    console.log(" --- error 3: " + JSON.stringify(err));
                });

                console.log(" --- D");

            }

            console.log(" --- E");

        });

        connection.commit(function (err) {
            if (err) {
                connection.rollback(function () {
                    console.log(" --- error 4: " + JSON.stringify(err));
                });

                console.log(" --- F");

            }

            console.log(" --- G");

            req.flash('success', 'Person was updated successfully.');
            res.redirect('/person/' + idperson);

            connection.release();
        });

    });

});

我得到以下输出序列:

 --- A
 --- C
 --- D
 --- E
 --- G
 --- error 3: {"code":"ER_DUP_ENTRY","errno":1062,"sqlState":"23000","index":0}

以下是我的证明:)

enter image description here

1 个答案:

答案 0 :(得分:1)

如果您希望异步函数按顺序运行,您必须将它们嵌套,或者将它们作为promises数组的一部分来调用,或者使用async / await样式代码......某些东西。你在这里做的是打电话

  1. pool.getConnection
  2. connection.beginTransaction
  3. connection.query& connection.query& connection.com同时发送所有内容
  4. 在我的头顶,我尝试这样的事情:

    router.get('/myfunc', async (req, res) => {
    
      try {
    
        const connection = await pool.getConnection();
    
        try {
    
          await connection.beginTransaction();
    
          // update statement
          const sql_update = `update course_person
                              set rescheduled_date = CURDATE()
                              where idperson = ?
                              and rescheduled_date is null`;
          await connection.query(sql_update, [idperson]);
    
          // insert statement
          const sql_insert = `insert into course_person (idcourse, idperson) VALUES (?, ?)`;
          await connection.query(sql_insert, [newidcourse, idperson]);
    
          // commit
          await connection.commit();
    
          req.flash('success', 'Person was updated successfully.');
          return res.redirect('/person/' + idperson);
    
        } finally {
          pool.releaseConnection(connection);       
        }
    
      } catch (err) {
        await connection.rollback();
        console.log(err);
      }
    
    });
    

    注意:我还没有测试过这段代码。但这就是我使用的模式。

    这将要求您使用promise-mysql而不是node-mysql,因为async / await语法需要promises才能使用。您还可以使用Bluebird来宣传node-mysql。此外,如果您的节点版本还不支持async / await,那么您需要使用像babel这样的东西来传播它。 (我在工作流程中使用了babel。)

    如果你想简单地嵌套异步调用,对于更传统的编码风格,你可以这样做:

    pool.getConnection(function (err, connection) {
    
      connection.beginTransaction(function (err) {
        if (err) {
          console.log(" --- error 1: " +  JSON.stringify(err));
          throw err;
        }
    
        connection.query('update course_person set rescheduled_date = CURDATE() where idperson = ? and rescheduled_date is null', [idperson], function (err, rows, fields) {
          if (err) {
            connection.rollback(function () {
              console.log(" --- error 2: " + JSON.stringify(err));
              throw err;
            });
          }
    
          connection.query('insert into course_person (idcourse, idperson) VALUES (?, ?)', [newidcourse, idperson], function (err, rows, fields) {
            if (err) {
              connection.rollback(function () {
                console.log(" --- error 3: " + JSON.stringify(err));
                throw err;
              });
            }
    
            connection.commit(function (err) {
              if (err) {
                connection.rollback(function () {
                  console.log(" --- error 4: " + JSON.stringify(err));
                  throw err;
                });
              }
    
              req.flash('success', 'Person was updated successfully.');
              res.redirect('/person/' + idperson);
    
              connection.release();
            });
          });
        });
      });
    });
    

    注意每个调用是如何嵌套在前一个调用的回调函数中的。这开始变得混乱所有的缩进(即"回调地狱")。使用promises或async / await的代码从长远来看更易读,更容易维护。