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);
// 此处我们想提交事务，释放锁；但是事与愿违
session
4000
1.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有一定帮助。