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");
        }
    }
}