JpaRepository执行save时触发意外更新，是否符合预期？

嗨，这确实是JPA（准确说是Hibernate实现的JPA）的预期行为，我来给你拆解背后的原因，以及怎么优化这个不必要的全量更新问题：

为什么会触发父实体+所有子实体的更新？ #

你的代码里有几个关键因素共同导致了这个情况：

• EAGER加载的关联：父实体ExchangeDbo的listCurrencyPair用了FetchType.EAGER，这意味着每次你加载父实体时，所有关联的CurrencyPairDbo子实体都会被立刻拉进持久化上下文里。
• Hibernate的脏检查机制：在@Transactional方法执行结束时，Hibernate会遍历持久化上下文里的所有实体，检查它们是否有变更。哪怕实体根本没修改，只要它在上下文里，结合@Version注解，Hibernate就可能生成更新语句（甚至递增版本号）。
• 级联操作+操作父实体：你调用的是exchangeRepository.save(exchangeDbo)，而父实体的@OneToMany设置了cascade = CascadeType.ALL——这会触发对所有关联子实体的级联操作，再加上脏检查，就导致所有已加载的子实体都被“更新”了一遍。

怎么优化这个行为？ #

1. 把关联切换为LAZY加载（最推荐） #

把关联的fetch属性改成FetchType.LAZY，这样只有当你主动访问子实体集合时，才会加载它们，大大减少持久化上下文里的实体数量：

// ExchangeDbo中的@OneToMany修改
@OneToMany(mappedBy = "exchange", cascade = CascadeType.ALL, orphanRemoval = true, fetch = FetchType.LAZY)
@BatchSize(size=100)
@Getter
private List<CurrencyPairDbo> listCurrencyPair = new ArrayList<>();

// CurrencyPairDbo中的@ManyToOne也改成LAZY
@ManyToOne(fetch=FetchType.LAZY)
@Getter
private ExchangeDbo exchange;

2. 直接保存子实体，而非父实体 #

既然你是新增子实体，其实完全可以直接调用子实体Repository的save()方法——因为子实体已经关联了父实体，JPA会自动维护两者的关联关系，没必要操作父实体：

@Transactional(readOnly=false, rollbackFor=Exception.class)
public DboReturns addNewCurrencyPair(ExchangeDbo exchangeDbo, String currencyPairString) {
    CurrencyPairDbo currencyPairDbo = exchangeDbo.addCurrencyPair(new CurrencyPair(currencyPairString));
    // 直接保存子实体即可
    CurrencyPairDbo savedPair = currencyPairRepository.save(currencyPairDbo);
    return new DboReturns(exchangeDbo, savedPair, null);
}

3. 用代理对象代替加载完整实体 #

如果不需要修改父实体，尽量用getReferenceById()获取父实体的代理对象，而不是find()——代理对象不会加载关联的子实体，能进一步减少上下文里的实体：

// 比如获取父实体时用这个
ExchangeDbo exchangeDbo = exchangeRepository.getReferenceById(exchangeId);

额外补充 #

当实体带有@Version注解时，Hibernate的脏检查会更严格：哪怕实体没有任何变更，只要它被加载进持久化上下文，有时候也会触发版本号递增和更新语句——这是乐观锁机制的一部分，用来保证并发场景下的数据一致性。切换为LAZY加载能从根源上减少这种不必要的更新操作。

内容的提问来源于stack exchange，提问作者Ian