Spring Data JPA调用save()未更新反而插入新记录问题排查

问题现象 #

从数据库获取已存在的User实体，直接调用save()方法（即使未修改任何属性），每次都会在数据库中新增一条记录，而非更新现有数据。测试用REST控制器代码如下：

@GetMapping("/user")
public void test() {
    User byEmail = userRepository.findByEmail("test@test.com").orElse(null);

    if (byEmail != null) {
        userRepository.save(byEmail);
    } else {
        User user = new User();
        user.setEmail("test@test.com");
        userRepository.save(user);
    }
}

相关代码与配置 #

User实体类 #

@Getter
@Setter
@Entity
@Builder
@AllArgsConstructor
@NoArgsConstructor
@ToString
@Table(name = "forum_user")
public class User extends BaseEntityAudit {

    @Id
    @GeneratedValue(generator = "uuid")
    @GenericGenerator(name = "uuid", strategy = "uuid")
    private String id;

    @Column(name = "first_name")
    private String firstName;

    @Column(name = "last_name")
    private String lastName;

    @Email
    @Column(name = "email", unique = true)
    private String email;

    @Column(name = "external_id")
    private String externalId;

    @Column(name = "picture_url")
    private String pictureUrl;

    @OneToMany(fetch = FetchType.EAGER)
    @JoinColumn(name = "user_id")
    @JsonIgnore
    private List<Post> posts;

    @ManyToMany(fetch = FetchType.EAGER)
    @JoinTable(
            name = "user_group",
            joinColumns = @JoinColumn(name = "user_id"),
            inverseJoinColumns = @JoinColumn(name = "group_id")
    )
    @JsonIgnore
    private List<Group> userGroups;

    @ManyToOne
    @JoinColumn(name = "role_id", referencedColumnName = "id", nullable = false)
    private Role role;

    @OneToMany(cascade = CascadeType.ALL, fetch = FetchType.EAGER)
    @JoinColumn(name = "author_id")
    @JsonIgnore
    private List<NameTag> nameTags;

}

Repository查询方法 #

Optional<User> findByEmail(String email);

用户表DDL #

create table C##USER2.FORUM_USER
(
    ID          VARCHAR2(32) not null
        constraint SYS_C008383
            primary key,
    FIRST_NAME  VARCHAR2(45),
    LAST_NAME   VARCHAR2(45),
    EMAIL       VARCHAR2(255),
    EXTERNAL_ID VARCHAR2(45),
    PICTURE_URL CLOB,
    ROLE_ID     VARCHAR2(32),
    CREATED_AT  TIMESTAMP(6),
    CREATED_BY  VARCHAR2(32),
    UPDATED_AT  TIMESTAMP(6),
    UPDATED_BY  VARCHAR2(32)
)
/

可能的原因分析 #

1. 父类BaseEntityAudit实现Persistable导致isNew判断逻辑异常 #

如果BaseEntityAudit实现了org.springframework.data.domain.Persistable接口，Spring Data JPA会依据该接口的isNew()方法判断实体是否为新对象，而非默认的ID是否为null。

• 例如：若isNew()方法基于createdAt字段是否为null判断，但查询到的实体createdAt被意外重置为null，或者isNew()逻辑错误地返回true，则即使实体已有ID，仍会被视为新对象执行persist（新增）操作。

2. 事务缺失导致实体处于Detached状态，merge操作异常 #

控制器的test()方法未添加@Transactional注解：

• findByEmail()查询完成后，实体立即脱离持久化上下文（变为Detached状态）。
• 调用save()时会执行merge操作，但如果关联实体（如Role）也处于Detached状态，且JPA无法正确合并关联关系，可能导致框架误判为新实体执行新增。

3. equals/hashCode方法未正确实现 #

若User或BaseEntityAudit未基于主键ID重写equals()和hashCode()方法：

• JPA在判断实体是否存在于持久化上下文时，可能无法识别查询到的实体与数据库中已有实体为同一对象，从而触发新增操作。

4. 实体类注解与数据库约束不一致 #

实体类中email字段标注了@Column(unique = true)，但数据库DDL中未添加对应的唯一约束：

• 这虽不是直接导致新增的原因，但会允许数据库中存在重复email的记录，间接混淆了框架对实体唯一性的判断。

排查建议 #

1. 检查BaseEntityAudit类是否实现Persistable接口，若实现则确认isNew()方法的逻辑是否正确，确保已有ID的实体返回false。
2. 在控制器的test()方法上添加@Transactional注解，让实体始终处于持久化上下文的托管状态，避免Detached状态导致的merge异常。
3. 重写User类的equals()和hashCode()方法，基于主键ID实现：

   @Override
   public boolean equals(Object o) {
       if (this == o) return true;
       if (o == null || getClass() != o.getClass()) return false;
       User user = (User) o;
       return Objects.equals(id, user.id);
   }
   
   @Override
   public int hashCode() {
       return Objects.hash(id);
   }
   

4. 修正数据库DDL，为EMAIL字段添加唯一约束，与实体类注解保持一致：

   ALTER TABLE C##USER2.FORUM_USER ADD CONSTRAINT FORUM_USER_EMAIL_UNIQUE UNIQUE (EMAIL);
   

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