mybatis中commit和rollback使用不当造成事务泄露和锁超时问题总结

commit接口定义如下：

  /**

   * Flushes batch statements and commits database connection.

   * Note that database connection will not be committed if no updates/deletes/inserts were called.

   * To force the commit call {@link SqlSession#commit(boolean)}

   */

  void commit();

  /**

   * Flushes batch statements and commits database connection.

   * @param force forces connection commit

   */

  void commit(boolean force);

在使用这两个函数不当时，可能会造成事务泄露和锁超时问题。

1. 事务泄露场景

注意commit()注释提示部分，打开一个不自动提交事务的session，如果这个session在commit之前没有update/delete/insert执行语句时，session.commit()是不会提交事务的，此时要想提交事务需要使用commit(true)方法，否则在session.close()之前这个事务是一直没有释放的。

2. 锁超时问题

也许你会说针对没有更新语句的场景我们可以开启自动提交事务的session来避免，但是当需要select for update语句时候你是必须要开启非自动提交事务的session的。此时如果不使用session.commit(true)，不仅会造成事务没有及时释放而且可能造成逻辑上的错误（锁超时问题）。

比如以下代码就会造成逻辑上的错误：

// 开启一个非自动提交事务session

SqlSession session1 = sessionFactory.openSession(false);

// select for update 语句锁住一条主键为1的记录

selectByIdForUpdate(session1, 1);

// 此处我们想提交事务，释放锁；但是事与愿违

session1.commit();

// 开启一个非自动提交事务session

SqlSession session2 = sessionFactory.openSession(false);

// 使用新的session锁住这条记录，此处会出现锁超时问题，

// 因为上个事务并没有真正的提交；这不是我们锁期望的

selectByIdForUpdate(session2, 1);

session2.commit();

// 关闭连接

session1.close();

session2.close();

rollback的使用和commit的类似，也应该注意这些问题。

综上所述：在使用commit和rollback的时候要注意这些问题，在没有确定session是否有更新语句的情况下，尽量使用commit(true)和rollback(true)函数。这些问题都是我们在工作过程中遇到了，仔细查看了源码和用户手册总结出的经验教训，希望对大家使用mybatis有一定帮助。