最近同事发现一个业务状态部分更新的bug,这个bug会导致两张表的数据一致性问题。花了些时间去查问题的原因,现在总结下里面遇到的知识点原理。
我们先看一段实例代码,来说明下问题:
@Service
public class PaymentServiceImpl implements PaymentService {
public void fetchLatestStatus(String trxId) {
//1. do RPC request and get the payment status
StatusResponse response = doRPC(trxId);
//2. save request data
saveRequest(response);
}
@Transactional(propagation = Propagation.REQUIRES_NEW, rollbackFor = Exception.class)
public void updatePayment(StatusResponse response) {
Payment pay = payRepository.findByTrxId(response.getTrxId);
//do something to update payment record by response and persist
pay.setStatus(success);
payRepository.save(pay);
}
}
在上面代理里,updatePayment
方法的@Transactional注解会失效,并没有新开一个事务去保存Payment
对象。
开发中少不了用到事务注解@Transactional
来管理事务,@Transactional注解底层是基于Spring AOP来进行实现的。
我们来看两个典型的AOP应用场景:
我们先复习下Spring AOP动态代理的原理。 AOP是一种通用的编程思想,Java里有2种实现方式:
Spring AOP
如上图所示 TransactionInterceptor (事务拦截器)在目标方法执行前后进行拦截,DynamicAdvisedInterceptor(CglibAopProxy 的内部类)的 intercept 方法或 JdkDynamicAopProxy 的 invoke 方法会间接调用 AbstractFallbackTransactionAttributeSource的 computeTransactionAttribute 方法,获取Transactional 注解的事务配置信息。
但是当发生方法内调用的时候,被调用的函数Class.transactionTask()
尽管看起来加了事务注解,但是并没有执行代理类对应的方法ProxyClass.transactionTask()
,导致注解跟没写一样。
把@Transactional
注解加在private
修饰的方法也会一样的现象,原理其实一样的。
搞清楚了原理,问题的原因就清晰了: 这个问题的原因从表面来说,是因为在同一个Class内,非代理增强方法中调用了被@Transactional注解增强的方法,注解会失效。背后的实际原因是Spring AOP是基于代理,同一个类内这样调用的话,只有第一次调用了动态代理生成的ProxyClass,之后调用是不带任何切面信息的方法本身,因为没有直接调用Spring生成的代理对象。
把updatePayment
方法放到另外一个类里,让Spring自动为其生成代理对象,调用方就能调用到updatePayment对应的ProxyObject的方法了。
我们还提到了AspectJ也是实现AOP的一种方式,那么AspectJ有这样的方法内调用失效问题吗?
可以关注**好奇心森林**公众号后台回复AOP,索取我总结的AOP思维脑图,答案就在里面
还是之前的一段代码,我们把updatePayment
方法放在一个单独的类里。会发现之前payRepository.save(pay)
必须显式声明保存,但是如果抽出来后就不用再写也能自动保存。
@Service
public class PaymentServiceImpl implements PaymentService {
@Transactional(propagation = Propagation.REQUIRES_NEW, rollbackFor = Exception.class)
public void updatePayment(StatusResponse response) {
Payment pay = payRepository.findByTrxId(response.getTrxId);
//do something to update payment record by response and persist
pay.setStatus(success);
//payRepository.save(pay);
xxxRepository.save(xxx);
}
}
这个区别需要知道Hibernet对Entity的状态管理机制,在Hibernet里一个对象有多种状态:
通过findByTrxId
查出来的Payment对象处于托管态,任何改变pay对象的操作比如pay.setStatus()都会在事务结束的时候自动提交。
另外同事发现一个有趣的区别:
在Controller调用PaymentServiceImpl.updatePayment()不需要显式保存pay对象,也能持久化到数据库,然而用Spring的定时器调用就不会生效。
经过Debug发现,Spring框架在每个request通过OpenEntityManagerInViewInterceptor
的preHandle
方法里为每个request都建了一个EntityManager, 具体参见Spring源码:
在Spring配置里加上spring.jpa.open-in-view=false
就会关闭每个request的EntityManager,通过controller调用就和定时器现象一样了。
Open Session In View简称OSIV,是为了解决在mvc的controller中使用了hibernate的lazy load的属性时没有session抛出的LazyInitializationException异常。
对hibernate来说ToMany关系默认是延迟加载,而ToOne关系则默认是立即加载;而在mvc的controller中脱离了persisent contenxt,于是entity变成了detached状态,这个时候要使用延迟加载的属性时就会抛出LazyInitializationException异常,而Open Session In View 旨在解决这个问题。
Tips:
通过OSIV技术来解决LazyInitialization问题会导致open的session生命周期过长,它贯穿整个request,在view渲染完之后才能关闭session释放数据库连接;另外OSIV将service层的技术细节暴露到了controller层,造成了一定的耦合,因而不建议开启,对应的解决方案就是在controller层中使用dto,而非detached状态的entity,所需的数据不再依赖延时加载,在组装dto的时候根据需要显式查询。
通过一个bug的例子,我们总结了:
如果觉得有所收获,麻烦帮我顺手点个在看吧,你的举手之劳对我来说就是最大的鼓励。 END~