转载

为什么JPA保存数据失败

最近同事发现一个业务状态部分更新的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，基于动态代理实现
- JDK代理
- Cglib代理
AspectJ，基于编译期实现

Spring实现AOP的方法则就是利用了动态代理机制实现的；
在应用系统调用声明@Transactional 的目标方法时，Spring Framework 默认使用 AOP 代理，在代码运行时生成一个代理对象ProxyObject，如：

整个事务的增强执行过程是这样的：为什么JPA保存数据失败

如上图所示 TransactionInterceptor （事务拦截器）在目标方法执行前后进行拦截，DynamicAdvisedInterceptor（CglibAopProxy 的内部类）的 intercept 方法或 JdkDynamicAopProxy 的 invoke 方法会间接调用 AbstractFallbackTransactionAttributeSource的 computeTransactionAttribute 方法，获取Transactional 注解的事务配置信息。

但是当发生方法内调用的时候，被调用的函数 Class.transactionTask() 尽管看起来加了事务注解，但是并没有执行代理类对应的方法 ProxyClass.transactionTask() ，导致注解跟没写一样。为什么JPA保存数据失败

把 @Transactional 注解加在 private 修饰的方法也会一样的现象，原理其实一样的。

搞清楚了原理，问题的原因就清晰了：这个问题的原因从表面来说，是因为在同一个Class内，非代理增强方法中调用了被@Transactional注解增强的方法，注解会失效。背后的实际原因是 Spring AOP是基于代理，同一个类内这样调用的话，只有第一次调用了动态代理生成的ProxyClass，之后调用是不带任何切面信息的方法本身，因为没有直接调用Spring生成的代理对象。

解决方法

把 updatePayment 方法放到另外一个类里，让Spring自动为其生成代理对象，调用方就能调用到updatePayment对应的ProxyObject的方法了。

思考

我们还提到了AspectJ也是实现AOP的一种方式，那么AspectJ有这样的方法内调用失效问题吗？

可以关注**好奇心森林**公众号后台回复AOP，索取我总结的AOP思维脑图，答案就在里面

问题二：定时器运行没启动em

还是之前的一段代码，我们把 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里一个对象有多种状态：

Transient 瞬时态：直接new出来的对象，既没有被保存到数据库中，也不处于session缓存中
Persistent 持久态：已经被保存到数据库中并且加入到session缓存中
Detached 游离态：已经被保存到数据库中但不处于session缓存中

通过 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的时候根据需要显式查询。