未提交/回滚Sequelize非托管事务的影响及相关问题咨询

背景 #

我们团队接手了一个由前供应商开发的遗留项目，负责维护代码及修复潜在Bug。该项目基于Node.js/Express、Sequelize（ORM）及PostgreSQL构建。梳理代码时发现一处非托管Sequelize事务存在问题：当检测到用户已存在时，未对事务执行commit()或rollback()操作，代码示例如下：

const createUser = async (req, res) => {
  const t = await db.transaction();
  try {
    const { phone, email, password } = req.body;
    
    const existedUser = await User.findOne({
      where: {
        [Op.or]: {
          phone: phone,
          email: email.toLowerCase(),
        },
        deleted_at: null,
      },
    });
    
    if (!existedUser){
      const newUser = await User.create(
        {
          phone: phone,
          email: email.toLowerCase(),
          password: hash(password),
        },
        { transaction: t }
      );
      
      await t.commit();
      res.status(200).json({
        isSuccess: true,
        statusCode: 200,
        responseMessage: "User created successfully",
        data: newUser,
      });
    } else {
      // The problem I'm wondering about is here
      // No transaction commit or rollback in this case
      res.status(400).json({
        isSuccess: false,
        statusCode: 400,
        responseMessage: "User already exist",
        data: null,
      });
    }
    
  } catch (err) {
    await t.rollback();
    res.status(500).json({
      isSuccess: false,
      statusCode: 500,
      responseMessage: err.message,
      data: null,
    });
  }
}

问题与解答 #

1. 非托管事务不手动提交或回滚是否合规？ #

完全不合规。Sequelize的非托管事务要求开发者手动控制生命周期：要么通过commit()提交变更，要么通过rollback()撤销所有操作。未执行这两个操作的事务会处于悬挂状态，既不完成也不终止，违反了事务ACID原则的原子性要求，也不符合Sequelize的事务使用规范。

2. 若应用中存在大量此类情况，会引发哪些后果？ #

• 数据库连接耗尽：PostgreSQL会为每个事务占用一个数据库连接，悬挂事务不会主动释放连接，当此类请求累积到一定数量时，会占满数据库连接池，导致新请求无法获取连接，服务彻底瘫痪。
• 数据锁定问题：如果事务中执行了加锁操作（比如带排他锁的查询、写操作），悬挂事务会一直持有锁，阻塞其他请求对相关数据的读写，引发严重性能问题甚至死锁。
• 事务日志膨胀：数据库会为每个事务维护日志，未完成的事务会持续占用日志空间，可能导致磁盘空间耗尽，影响数据库正常运行。

3. 此类未完成事务是否会浪费资源、拖慢后端应用？ #

会的。

• 数据库资源浪费：悬挂事务占用数据库连接、内存、锁资源，这些资源无法被其他请求复用，直接降低数据库的处理能力。
• 后端应用性能下降：当数据库连接池被占满后，后端应用的新请求会处于等待连接的状态，响应时间急剧增加，吞吐量大幅下降，最终导致应用服务变慢甚至不可用。

内容的提问来源于stack exchange，提问作者masadamsahid