JPA报错：启用delete-orphan后集合引用无法修改问题排查

报错信息 #

Caused by: org.hibernate.HibernateException: Don't change the reference to a collection with delete-orphan enabled : com.asdf.data.entity.jpa.impl.Account.authorizations

相关代码片段 #

报错触发的保存方法 #

@Override
public Account save(Account account) {
    if (account == null) {
        return null;
    } else {
        String normailzedUserDN = UserDNParseUtil.normalizeUserDN(account.getUserDn());
        cacheConfig.evictSingleCacheValue("accountCache", normailzedUserDN);
        
        Session session = em.unwrap(Session.class);
        session.save(account);
        
        String query = "select a from Account a where a.isActive IS TRUE and a.userDn = :userDn";
        
        TypedQuery<Account> q = em.createQuery(query, Account.class);
        q.setParameter("userDn", account.getUserDn());
        
        account = q.getSingleResult(); // 报错发生在第80行
        
        return account;
    }
}

Account类集合映射代码 #

@JsonIgnore
@OneToMany(cascade = CascadeType.ALL, orphanRemoval=true, mappedBy = "account", fetch = FetchType.EAGER)
private Set<Authorization> authorizations;

@Valid
public Set<Authorization> getAuthorizations() {
    return authorizations;
}

public void setAuthorizations(final Set<Authorization> authorizations) {
    if (this.authorizations==null) {
        this.authorizations=new HashSet<Authorization>();
    } else {
        this.authorizations.clear();
    }
    this.authorizations.addAll(authorizations);
}

报错原因 #

当启用orphanRemoval=true时，Hibernate会严格监控集合的对象引用：它要求持久化上下文中的集合实例必须是最初关联的那一个，绝对不能被替换成新的集合对象。

你的问题核心在于：

1. 执行session.save(account)后，原account实例已经进入持久化上下文；
2. 后续的查询q.getSingleResult()返回了一个全新的Account实例，这个实例的authorizations是Hibernate新创建的集合对象（比如PersistentSet）；
3. 你把原account变量指向了这个新实例，导致持久化上下文中的原实例与新实例的集合引用不一致，Hibernate在清理上下文时判定为“集合引用被修改”，触发了异常。

另外FetchType.EAGER会强制Hibernate立即加载集合，进一步放大了引用不一致的冲突概率。

解决方案 #

方案1：避免替换持久化实例引用 #

不要将查询结果赋值给原account变量，而是复用原实例，同步必要字段即可：

// 替换原代码中的account = q.getSingleResult();
Account dbAccount = q.getSingleResult();
// 仅同步需要更新的字段，比如自增ID、数据库生成的属性
account.setId(dbAccount.getId());
// 其他需要同步的字段按需添加

方案2：改用merge()替代save()+查询 #

session.merge()会自动将传入的实体状态合并到持久化上下文的实例中，并返回持久化后的实例，无需额外查询：

// 替换原代码中的session.save(account)及后续查询逻辑
account = (Account) session.merge(account);

方案3：优化集合访问（辅助措施） #

可以修改getter返回不可修改集合，防止外部意外替换集合引用：

public Set<Authorization> getAuthorizations() {
    return Collections.unmodifiableSet(authorizations);
}

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