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.

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

Feat/comment pagination #172

Open
wants to merge 3 commits into
base: feat/cookie
Choose a base branch
from

Conversation

chjcode
Copy link
Contributor

@chjcode chjcode commented Sep 14, 2024

#️⃣연관된 이슈

아직 작업중입니다!

📝작업 내용

1. Comment entity 변경 사항

  • group entity 추가
  • assignGroup추가

2. Comment Service 변경 사항

부모 댓글인 경우 자신의 id값을 가져야 하므로 saveAndFlush 후 id값을 가진 후에 group의 값을 변경하도록 합니다.

save()메서드의 경우 @transactional로 관리되고 있습니다. Hibernate의 경우, SQL연산을 즉시 수행하지 않고, 트랜잭션 커밋 시점이나 플러시가 발생할 때까지 연기합니다.(Hibernate의 쓰기 지연 전략). 따라서 그냥 commentDAO.save(comment)를 하게 되면 group에는 null값이 담기게 됩니다. 따라서 flush()를 사용하여 즉시 INSERT 하여 지금까지의 변경 사항을 데이터베이스에 즉시 반영하도록 한 후에 group에 id값을 넣어주도록 했습니다.

3. CommentResponseDTO 변경 사항

  1. group 값을 반환하도록 추가
  2. 삭제된 댓글이 comment_id, group, created_at값은 null이 아닌 값으로 response해주도록 변경

- response에 group 추가
- keyset pagination 적용을 위해 delete = true 인 댓글이 createdAt값과 group값을 가지도록 변경'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant