两个service互相调用，抛出异常时的事务处理

Question

问题描述

serviceA中的methodA调用serviceB中的methodB，methodB抛出一个异常，然后methodA捕获掉这个异常，就会出现
Transaction rolled back because it has been marked as rollback-only 异常
(serviceA和serviceB都加了注解@Transactional(rollbackFor = Exception.class))

问题出现的环境背景及自己尝试过哪些方法

业务需求是在serviceA中的methodA中调用serviceB中的methodB，但是methodB有一定几率会抛异常，methodA要把异常捕获，并做其他处理。
但是methodA返回结果的时候就会报“Transaction rolled back because it has been marked as rollback-only”异常。
解决办法是在methodB上添加注解@Transactional(rollbackFor = Exception.class, propagation = Propagation.REQUIRES_NEW)。还有一种办法是把methodB写到serviceA中，直接用this.methodB调用

疑惑

现在想搞明白为什么serviceA不能捕获serviceB中的异常？
serviceA中的methodA中调用serviceB中的methodB和同一个service中两个方法调用，在事务处理上有什么区别？

Answer 1

不是不能捕获，你在serviceA中确实捕获了serviceB中的异常，只是异常捕获处理之后事务进行提交之前进行了检查从而又抛出了异常，简单分析下题主的场景：

因为spring的默认传播属性，serviceB的事务加入到serviceA的事务中，二者在同一个事务中，假设在serviceB中发生了异常（比如字段长度100而内容长度150），你在serviceA中捕获了该异常，所以事务没有进行回滚，当serviceA的事务进行提交之前，因为serviceB发生过SQL异常，导致整个事务标记为不可提交状态（通过serviceB的代理方法标记），所以重新抛出
Transaction rolled back because it has been marked as rollback-only异常。SQL本来就是错的当然不能提交咯。当你把传播属性改为Propagation.REQUIRES_NEW,两者就不在同一事物内了，也就是说serviceB的错误sql不会影响serviceA的事务。
第二个问题的答案，你看看这个https://segmentfault.com/q/10...

Answer 2

这个和String的AOP有关。具体细节只有看spring代码才知道怎么回事。但如果你加入注解@Transactional之后，就有可能被spring的AOP处理，spring可能会在你抓住异常之前就拦截了异常，然后做自己的处理。

所以一个service类的方法最好不要调用被@Transactional注解的方法。这样你会搞不清楚spring会如何处理，spring自己也没有说明白甚至可能会很随意的处理。

最好是把methodB里面需要被methodA调用的部分提取出来，单独放在一个utils类中，不加入spring的注解，然后由methodB、methodA调用，避免methodA调用被@Transactional注解的方法。