Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[고도화] 댓글 기능 고도화 - 1차 대댓글: 도메인 업데이트 #62

Closed
Tracked by #61
kiekk opened this issue Oct 3, 2023 · 1 comment
Closed
Tracked by #61

[고도화] 댓글 기능 고도화 - 1차 대댓글: 도메인 업데이트 #62

kiekk opened this issue Oct 3, 2023 · 1 comment
Assignees
@kiekk
Labels
enhancement New feature or request
Milestone
게시판 서비스 고도화

Comments

@kiekk
Copy link
Owner

kiekk commented Oct 3, 2023 •
edited
Loading

대댓글 기능을 위한 도메인 정보를 업데이트 한다.

  • ArticleComment 에 parentCommentId 추가
@kiekk kiekk mentioned this issue Oct 3, 2023
Closed
4 tasks
@kiekk kiekk self-assigned this Oct 3, 2023
@kiekk kiekk added the enhancement New feature or request label Oct 3, 2023
@kiekk kiekk added this to the 게시판 서비스 고도화 milestone Oct 3, 2023
@kiekk kiekk changed the title 도메인 업데이트 [고도화] 댓글 기능 고도화 - 1차 대댓글 - 도메인 업데이트 Oct 3, 2023
@kiekk kiekk changed the title [고도화] 댓글 기능 고도화 - 1차 대댓글 - 도메인 업데이트 [고도화] 댓글 기능 고도화 - 1차 대댓글: 도메인 업데이트 Oct 3, 2023
@kiekk
Copy link
Owner Author

kiekk commented Oct 3, 2023 •
edited
Loading

대댓글 기능 구현시 단방향 관계를 사용해보았는데 에러가 발생했다.

현재 환경은

  • Spring Boot 3.1.3
  • Hibernate-core 6.2.7 Final

찾아보니 Spring Boot 3.1.X 과 Hibernate-core 6.2.X 에서 발생하는 Hibernate 버그 같다.

관련 링크

해결 방법

  • 가장 간단하게는 Spring Boot를 3.1 보다 아래 버전으로 사용하는 것이지만 이 방법은 다른 의존성과의 버전과 변경점도 파악해야 하기 때문에 좋은 방법은 아니다.
  • 내가 해결한 방법은 Hibernate에서 이 버그를 픽스한 버전을 사용한 것이다.

이 버그가 적용된 버전은
6.3.0.CR1이며 FINAL에 적용된 버전은 6.3.1.FINAL을 사용하면 된다.

  • implementation 'org.hibernate.orm:hibernate-core:6.3.0.CR1'
  • implementation 'org.hibernate.orm:hibernate-core:6.3.1.Final'

kiekk added a commit that referenced this issue Oct 3, 2023
@kiekk
#62 - 대댓글 도메인 설계
a6a4e0f 
대댓글 도메인 안에서 부모, 자식 관계를 설정하는 코드를 추가
자식 댓글의 컬렉션 변화가 쿼리에 반영되게끔
cascading 규칙을 ALL로 적용
이번엔 단방향 연관관계 설정을 사용해봄
따라서 부모 댓글은 엔티티가 아닌 `Long` id를 직접 표현

또한 자식 댓글을 추가할 수 있는 메서드 추가 제
@kiekk kiekk mentioned this issue Oct 3, 2023
Merged
kiekk added a commit that referenced this issue Oct 3, 2023
@kiekk
#62 - 대댓글 도메인 설계
7ca375b 
대댓글 도메인 안에서 부모, 자식 관계를 설정하는 코드를 추가
자식 댓글의 컬렉션 변화가 쿼리에 반영되게끔
cascading 규칙을 ALL로 적용
이번엔 단방향 연관관계 설정을 사용해봄
따라서 부모 댓글은 엔티티가 아닌 `Long` id를 직접 표현

또한 자식 댓글을 추가할 수 있는 메서드 추가 제
@kiekk kiekk closed this as completed Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kiekk kiekk

Labels
enhancement New feature or request
Projects
None yet
Milestone
게시판 서비스 고도화
Development

No branches or pull requests
1 participant
@kiekk