你完全说对了——直接调用自动生成的updateTable1和updateTable2会触发两个独立的事务，一旦第二个更新失败，第一个已经提交的更新无法回滚，这确实不符合数据原子性的要求。要实现两个更新在同一个事务里执行，你需要编写一个自定义方法，把两个操作包裹在同一个事务上下文里。

先纠正下自动生成代码里的一个小笔误：updateTable2方法里的table1 = em.merge(table2);应该改成table2 = em.merge(table2);，不然会出现变量不匹配的编译问题。

接下来是正确的实现方式，我们可以写一个新的方法，统一管理事务和EntityManager：

@Action(Action.ACTION_TYPE.UPDATE)
public void updateBothTables(Table1 table1, Table2 table2) throws Exception {
    EntityManager em = getEntityManager();
    try {
        utx.begin(); // 开启单个事务
        em.joinTransaction();
        
        // 执行两张表的更新操作
        table1 = em.merge(table1);
        table2 = em.merge(table2);
        
        utx.commit(); // 只有两个操作都成功才提交
    } catch (Exception ex) {
        try {
            if (utx != null && utx.getStatus() == Status.STATUS_ACTIVE) {
                utx.rollback(); // 任何一个操作失败，回滚整个事务
            }
        } catch (Exception e) {
            ex.addSuppressed(e); // 保留原始异常信息，方便排查问题
            throw e;
        }
        throw ex;
    } finally {
        if (em != null && em.isOpen()) {
            em.close(); // 最后统一关闭EntityManager
        }
    }
}

关键要点说明： #

• 单个事务上下文：整个方法只调用一次utx.begin()和utx.commit()，确保两个更新操作处于同一个事务中，要么全部成功提交，要么全部失败回滚。
• 共享EntityManager：使用同一个EntityManager处理两个实体的merge操作，避免多个EntityManager带来的持久化上下文不一致问题。
• 原子性保障：只有当两个merge操作都成功完成时，才会提交事务；任何一步抛出异常，都会触发整个事务的回滚，保证两张表的数据一致性。
• 异常处理优化：添加了事务状态检查，避免在事务已经结束时尝试回滚；同时用addSuppressed保留原始异常信息，不会丢失报错的关键线索。

如果你后续有更多类似的多表操作需求，也可以考虑把事务管理的逻辑抽成一个通用的工具方法，减少重复代码的编写。

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