Hard Delete 시 SiteUser를 간접 참조하는 엔티티는 어떻게 삭제할 것인가

[whqtker]

Ref

이슈
PR

양방향 연관관계

장점

• 조회 시 이점이 있다.
  • 현재 Post와 Comment는 양방향 연관관계가 설정되어 있는데, 그렇다면 post.getCommentList() 와 같이 객체 그래프 타고 자연스러운 흐름으로 데이터 조회 가능
• Cascade나OrphanRemoval 옵션을 통해 연관 객체까지 삭제 처리 가능
• JPQL이나 Fetch Join을 사용하면 하나의 쿼리로 연관 객체의 데이터까지 가져올 수 있음

단점

• 너무나 유명한 ... N+1 문제 -> 성능 저하 이슈
• 향후 MSA 분리 시 귀찮음 (현재로서는 문제 X)

간접 참조

장점

• 결합이 느슨하다 -> 다른 도메인의 변경에 영향을 덜 받는다.
• 특정 객체를 조회할 때 다른 객체까지 가져오는 N+1 문제가 발생하지 않는다.

단점

• 연관 데이터가 필요할 때 ID를 통해 추가로 가져와야 한다.

결론

• 반드시 같이 연관되어 조회되는 게 아니라면 양방향 연관 관계를 끊자

Hard Delete 시 발생하는 문제

발생한 문제

SiteUser 와 연관 객체 간 양방향 연관 관계를 끊었기에, SiteUser 객체 삭제 시 연관 객체의 데이터가 삭제되지 않은 문제 발생 + SiteUser 또한 삭제 안 됨(연관 데이터부터 삭제해야 하므로)

해결 방법

• SiteUser와 연관된 FK에 대해 ON DELETE CASCADE 옵션 추가하여 삭제되도록
  • 단, 꼬리를 무는 연관 관계 객체 데이터는 삭제 안 됨.
  • Post 는 이미지까지 삭제해야 함. FK 제약 조건으로만 삭제하면 이미지는 고아가 됨. 또한 Post 경우 Redis 캐시 데이터는 정리 안 됨
  • 애플리케이션 로직 - DB 간 불일치 발생
• 모든 FK에 ON DELETE CASCADE 적용하기
  • 애플리케이션 로직 - DB 간 불일치 발생
• 서비스에 삭제 로직 구현 후, 스케쥴러에서 사용
  • 애플리케이션 로직이 길어짐
  • SiteUser를 간접 참조하는 엔티티를 생성하는 경우 까먹지 않고 삭제 로직 구현 & 스케줄러에 작성해주어야 함.

[sukangpunch]

제 생각은 on delete cascade 를 셋팅해서, 유저 삭제시 연관 데이터들을 삭제하는 것 보다, 비즈니스 로직으로 유저 데이터를 삭제하는 것이 좋을 것 같다는 생각입니다.
이유는 다음과 같습니다.

1. 엔티티에서 나타내는 규칙과, db의 규칙이 달라짐
- 엔티티만 보고서는, 해당 엔티티가 삭제되었을 때, 어디까지 삭제에 영향이 갈 지 명확하지 않을 수 있다.

2. Post 삭제 시, 비즈니스 로직이 필수적으로 보임
- 게시글을 삭제 할 때, 게시글 뿐만 아니라 이미지, 조회수 도 삭제 해야 하는데, 해당 부분에선 비즈니스 로직이 강제되는 것처럼 보입니다. (Redis 캐시, S3 데이터 등)

3. comment 삭제의 복잡함
- 댓글을 삭제 할 때는, 대댓글은 경우, 부모 댓글과의 의존성 등이 있는데, cascade 로 삭제하게 된다면 해당 조건부 로직을 적용할 수 없습니다.

4. 채팅 데이터들 관련한 문제점
- 유저 삭제로 인해 채팅 방이 사라지면, 채팅 대상자는 어떻게 보여야 할지
- 반대로 유저 삭제로 인해 채팅 대상자 및 채팅 내역이 사라지면 어떻게 보여야 할지
- 채팅 이미지 관련 문제(post와 동일)

이와 같은 문제를 생각 했을 때, 삭제 시 비즈니스 로직이 강제 된다 생각합니다.
허나, 수 많은 연관관계를 생각하면다면, 이 모든걸 다 자바 코드로 처리하는 것에서 너무 복잡도가 올라가는 것이 아닌가 하는 우려도 존재합니다.