spring Transaction rolled back because it has been marked as rollback-only

  1. 类TransactionTest1 :
public class TransactionTest1 {
      
    @Autowired
    private TransactionTest2 transactionTest2;
    // 给外层方法加上事务注解
    @Transactional
    public void test(){
        try {
            transactionTest2.test();
        }catch (Exception e){
            logger.error(e.getMessage(),e);
        }
    }
}

类TransactionTest2 :

public class TransactionTest2 {
      
   
    // 给外层方法加上事务注解
    @Transactional
    public void test(){
        throw new ApplicationException("test");
    }
}

测试结果:

org.springframework.transaction.UnexpectedRollbackException: Transaction rolled back because it has been marked as rollback-only

分析: 1.spring的事务边界是在调用业务方法之前开始的,业务方法执行完毕之后来执行commit or rollback(Spring默认取决于是否抛出runtime异常).
如果抛出runtime exception 并在你的业务方法中没有catch到的话,事务会回滚。
一般不需要在业务方法中catch异常,如果非要catch,在做完你想做的工作后(比如关闭文件等)一定要抛出runtime exception,否则spring会将你的操作commit,这样就会产生脏数据.所以你的catch代码是画蛇添足。相当于Spring事务策略失效了。try-catch捕获了异常后,这种业务方法也就等于脱离了spring事务的管理,因为没有任何异常会从业务方法中抛出,全被捕获并“吞掉”,导致spring异常抛出触发事务回滚策略失效。

解决方法1: 如果在catch代码块中采用页面硬编码的方式使用spring api对事务做显式的回滚,这样写也未尝不可。

TransactionAspectSupport.currentTransactionStatus().setRollbackOnly(); //手动回滚,这样上层就无需去处理异常了

修改后代码:

public class TransactionTest1 {
      
    @Autowired
    private TransactionTest2 transactionTest2;
    // 给外层方法加上事务注解
    @Transactional
    public void test(){
        try {
            transactionTest2.test();
        }catch (Exception e){
            logger.error(e.getMessage(),e);
            TransactionAspectSupport.currentTransactionStatus().setRollbackOnly();
        }
    }
}

解决方法2:
将异常再往上抛

public class TransactionTest1 {
      
    @Autowired
    private TransactionTest2 transactionTest2;
    // 给外层方法加上事务注解
    @Transactional
    public void test(){
        try {
            transactionTest2.test();
        }catch (Exception e){
            logger.error(e.getMessage(),e);
            throw new RuntimeException("throw test exception");
        }
    }
}
原文地址:https://www.cnblogs.com/wanthune/p/13839446.html