事务不回滚

Question

我调用了两种方法，第一种是更新表，另一种是在另一个表中插入记录。 当第二个事务失败时， EJB不会回滚第一个事务。

这是我的支持 bean：

@ManagedBean
@ViewScoped
public class TransactionTestBean implements Serializable {

    @EJB
    private TransactionTestService service;

    public String loadView() {
        return "/test/transactionTest";
    }

    public void test() {
        try {
            service.updateTest();
        } catch (Exception e) {
        }
    }
}

EJB 接口：

@Local
public interface TransactionTestService {

    void updateTest() throws CustomException;
}

EJB 类：

@Stateless
@TransactionManagement
public class TransactionTestServiceImpl implements TransactionTestService {

    @Resource(mappedName = "java:jboss/datasources/xxxxxDS", shareable = true)
    public DataSource dataSource;

    private TransactionTestDAO dao;

    @PostConstruct
    public void init() {
        dao = new TransactionTestDAOImpl();
    }

    @PreDestroy
    public void destroy() {
        dao = null;
    }

    @Override
    @TransactionAttribute(TransactionAttributeType.REQUIRED)
    public void updateTest() throws CustomException {

        try (Connection connection = dataSource.getConnection()) {
            dao.updateRecord(connection);
            // dao.saveRecord(connection);
        } catch (SQLException exception) {
            throw new CustomException(exception, exception.getMessage());
        }
    }
}

而我的自定义例外：

@ApplicationException(rollback = true)
public class CustomException extends Exception {

    public CustomException(Throwable cause, String message) {
        super(message, cause);
    }
}

编辑：

添加了 DAO 类：

public class TransactionTestDAOImpl implements TransactionTestDAO {

    @Override
    public void updateRecord(Connection connection) throws CustomException {

        PreparedStatement preparedStatement = null;

        try {
            preparedStatement = connection.prepareStatement("UPDATE table_x SET field_x = ? WHERE field_y = 1");
            preparedStatement.setInt(1, 1);
            preparedStatement.executeUpdate();
        } catch (Exception exception) {
            throw new CustomException(exception, exception.getMessage());
        } finally {
            if (preparedStatement != null) {
                try {
                    preparedStatement.close();
                } catch (SQLException sqlException) {
                }
            }
        }
    }
}

和 DAO 接口：

public interface TransactionTestDAO {

    void updateRecord(Connection connection) throws CustomException;
}

Answer 1

这种情况下的关键问题是某些 JBoss 版本中数据源的默认设置错误。 原始代码很好，并且可以在其他应用程序服务器（WebSphere App Server 和轻量级 WebSphere Liberty）中正常工作。

在 JBoss 中创建的数据源不是 JTA - 在管理控制台中，未选中Use JTA设置，在 xml 相关设置中是<datasource jta="false" ... 。 将此设置更改为true解决了问题。 （JohnB，你写的定义 xa-datasource 修复了这个问题，但由于我没有看到你的原始 xml 和数据源定义，我相信在更改数据源期间你也更改了这个有缺陷的 jta="false" 设置）。 正如 Grzesiek 测试的那样，它也适用于非 xa 数据源。

这是一个非常糟糕的默认设置，因为它会导致容器无法管理事务，并导致 EJB组件中获得的连接中出现有缺陷的事务行为。

非常感谢Grzesiek D.帮助我诊断这个问题。

Answer 2

请试试这个

 
 
  
  @Override public void updateTest() throws CustomException { Connection connection = dataSource.getConnection(); try { connection.setAutoCommit(false); // this should be the key dao.updateRecord(connection); dao.saveRecord(connection); connection.commit(); } catch(Exception ex) { connection.rollback(); throw new CustomException(ex, ex.getMessage()); } finally { if(connection != null) { connection.close(); } } }
 
 

更新

我上面的回答有一个错误，因为上面的代码假设使用了BMT （ Bean-Managed Transactions ）。 但正如我们所看到的，您正在使用CMT （容器管理事务）。 因为@TransactionManagement相当于@TransactionManagement(TransactionManagementType.CONTAINER) ）。

上面的代码片段仅适用于BMT 。 使用CMT你应该得到如下错误：

Caused by: java.sql.SQLException: You cannot set autocommit during a managed transaction!

但是，我的错误最终变成了好事，因为当你写

这很好用（...）

然后我们找到了一个答案：您认为您的 EJB bean使用 CMT和 JTA，但由于某些错误，它没有。

---

在下面的评论中，我还建议您使用 JPA，但在这种简单的情况下，JDBC 就足够了。 CMT 事务也可以与 JDBC 一起自由使用。

此外，数据源的类型在这里无关紧要。 CMT 可以自由地与非 XA 数据源（也称为本地数据源）和 XA 数据源一起使用。

更新 2

用户@Gas 解决了以下评论中的问题。 为他点赞。

基本上：原始代码没有任何问题。 问题在于数据源的配置（必须启用JTA ）。 因此，通过 JBoss 管理控制台编辑数据源配置并设置复选框“使用 JTA ”。

Answer 3

我几乎可以肯定您的问题是由于您使用new关键字手动创建 DAO 引起的：

@PostConstruct
public void init() {
    dao = new XxxxDAOImpl();
}

当您执行此类操作时，您的 ejb 容器无法管理该对象生命周期和事务边界。 您应该让容器创建和管理 dao（并为您注入它）。 这样，您将在您的所有 EJB 方法中获得正确的事务传播——因此您的问题将得到解决。

为此，您可以简单地使用@Stateless注释您的 DAO 类，并将其注入您的TransactionTestServiceImpl类中：

@EJB
private XxxxDAO dao;

然后，当然，删除init和destroy方法。

个人建议

为什么要使用单独的附加 dao 层？ 在 Java EE 世界中，最方便的选择是使用EntityManager 。 实体管理器在大多数用例中很好地扮演了 dao 的角色。 尽管您示例中的 JDBC 已经足够好，但 JPA 更简单（恕我直言）。

更新

这是一个错误的猜测，请参阅我在此页面上的其他答案以及更新。

Answer 4

我认为问题在于 Connection/DataSource 根本不是您当前事务的一部分。 我建议不要注入 JDBC 连接：

1. 在 JTA 类型的persistence.xml中为定义的 DataSoruce 创建一个 PersistenceUnit。
2. 在您的 EJB 中注入相应的EntityManager 。
3. 从步骤 2 中注入的EntityManager解包Connection 。没有标准方法，请查看此答案。