派生existsBy查询致托管实体意外刷新的问题及方案咨询

在基于Spring Boot、Hibernate、MSSQL的REST Web服务中，编写了如下代码：

@Entity
class Puppy {
    private Long id;
    private String collarName;
    private String collarId;
}

@Service
class CollarService {

    private final PuppyService puppyService;

    @Transactional
    void updateCollar(Long puppyId, Collar changes) {
        // 执行一些操作
        Puppy puppy = puppyService.getOne(puppyId);
        
        puppy.setCollarName(changes.getCollarName());
        puppy.setCollarId(changes.getCollarId());
        
        puppyService.updatePuppy(puppyId, puppy);
    }
}

@Service
class PuppyService {
    private final PuppyRepository repository;
    
    Puppy getOne(Long puppyId) {
        return repository.findById(puppyId).orElseThrow(() -> new ResourceNotFoundException(Puppy.class, puppyId));
    }

    @Transactional
    void updatePuppy(Long puppyId, Puppy updatedPuppy) {
        // 执行一些操作
        Puppy databasePuppy = getOne(puppyId);
        
        if(repository.existsByCollarId(updatedPuppy.getCollarId())) {
            throw new PuppyCollarException();
        }
    
        // 执行一些操作
    }
}

预期调用updateCollar方法传入已被占用的collarId时，会抛出PuppyCollarException以转换为用户可读的异常信息；但实际触发DataIntegrityViolationException，只能返回通用错误提示。推测原因是CollarService获取的Puppy是托管实体，调用existsByCollarId查询时触发了实体变更的提前刷新。

目前有两种临时解决方案：

• 将updatePuppy方法的事务传播属性设置为REQUIRES_NEW：

@Transactional(propagation = Propagation.REQUIRES_NEW)
void updatePuppy(Long puppyId, Puppy updatedPuppy) {
    //...
}

此时托管实体不在同一事务中，不会被提前刷新；

• 在传递Puppy实体给PuppyService前将其分离。

咨询以下两个问题：

1. 为何调用派生的existsBy查询方法会导致托管实体提前刷新？
2. 哪种解决方案更优？使用REQUIRES_NEW方案是否存在潜在问题？

1. 为什么existsBy查询会触发托管实体提前刷新？ #

Hibernate会话在执行查询语句时，为保证查询结果的一致性，会自动触发flush操作，将当前会话中所有托管实体的变更同步到数据库。这是因为如果不先刷新变更，查询可能读取到旧数据，导致结果不准确。

你在CollarService的事务中修改了Puppy实体属性，此时该实体处于托管状态（由Hibernate会话管理）。当调用existsByCollarId派生查询时，Hibernate会先执行flush，把修改后的Puppy数据写入数据库——这时数据库层面的唯一约束（假设collarId有唯一索引）会被触发，直接抛出DataIntegrityViolationException，你的业务校验逻辑（existsBy判断）还没来得及生效。

2. 哪种解决方案更优？REQUIRES_NEW有潜在问题吗？ #

方案对比 #

• 分离实体方案：通过EntityManager.detach(puppy)将实体从会话中分离，后续查询不会触发flush该实体的变更。优势是保持事务一致性，所有操作在同一事务中，避免跨事务状态不一致；缺点是需要手动管理实体状态，多了一步分离操作。
• REQUIRES_NEW方案：新事务的会话与原事务独立，原会话的托管实体变更不会被新事务感知，查询时不会触发flush原实体变更，业务校验可正常执行，但存在明显潜在问题：
  • 事务隔离问题：新事务和原事务独立，可能出现数据不一致，比如原事务修改的其他数据，新事务查询时无法读取；
  • 数据一致性风险：若原事务后续回滚，新事务已提交的操作无法回滚，导致数据永久不一致；
  • 性能开销：每次调用updatePuppy都会开启新事务，增加数据库事务的创建和提交成本。

最优选择 #

优先选择分离实体方案，或更进一步优化代码逻辑：不要传递整个托管实体到updatePuppy，而是只传递需要修改的字段，直接基于数据库查询到的databasePuppy进行修改和校验，从根源上避免托管实体提前flush的问题。

优化后的updatePuppy示例：

@Transactional
void updatePuppy(Long puppyId, String collarName, String collarId) {
    Puppy databasePuppy = getOne(puppyId);
    
    if(repository.existsByCollarId(collarId)) {
        throw new PuppyCollarException();
    }

    databasePuppy.setCollarName(collarName);
    databasePuppy.setCollarId(collarId);
    // 无需手动update，Hibernate会在事务提交时自动同步
}

这种方式既避免了托管实体状态干扰，代码逻辑更清晰，还减少了不必要的实体传递。

内容的提问来源于stack exchange，提问作者D.Tomov