Skip to content

Conversation

@CodeVac513
Copy link
Collaborator

🔨 테스크

Issue

recent로 최근 게시글을 가져올 때, comments 값에 대한 일관성이 보장되어야 하는가?

예를 들어서 어떤 인기 블로그 글이 올라온지 1초만에 댓글이 달리고
지구 반대편 인터넷이 느린 곳에 사는 사람이 recent를 늦게 받으면 댓글 수와 redis에 캐싱된 값이 다를 수는 있겠더라고요.

하지만 예시와 같은 극단적인 상황이 거의 생기지 않을 것 + 댓글 수가 다르다고 치명적인 문제는 발생하지 않을 것, 두 가지의 이유로 Redis와 MySQL을 확인하며 최신 글의 comments 동기화에 대한 필요성이 없다고 느꼈습니다.
그리고 트렌드 포스트의 sse는 Redis에 feedId만 캐시하고 mysql에서 값을 가져오기 때문에 동기화 문제는 없을 것이라 생각했습니다.

📋 작업 내용

  • feed crawler에서 크롤링 시 comments 값을 0으로 초기화해서 Redis에 캐싱하도록 수정

📷 스크린 샷(선택 사항)

  1. 실제 블로그 업데이트된 포스트가 없어서 더미 데이터를 직접 저장했을 때 아래와 같이 정상적인 동작을 확인했습니다.
image
  1. 저번 작업에서 sse를 통해 comments 값이 반환되는 것을 첨부하지 않아서 이번 PR에 해당 동작 결과를 첨부했습니다.
image

@CodeVac513 CodeVac513 self-assigned this Aug 27, 2025
@CodeVac513 CodeVac513 added the ✨ Feature 기능 구현 label Aug 27, 2025
Copy link
Member

@asn6878 asn6878 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

new 키워드쪽 담당하신 @jungmyunggi 님이랑 이야기를 좀 해보고, 이쪽 상황을 조금 정확하게 파악해보면 좋을 것 같네요.

일단 이번 PR 자체는 comments의 초기값 자체를 추가하는것으로 보이기에 현 안건이랑은 크게 상관없을 것 같다는 의견입니다!

@CodeVac513 CodeVac513 requested a review from Jo-Minseok August 27, 2025 07:31
Copy link
Member

@Jo-Minseok Jo-Minseok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다!
본 테스크는 제가 언급을 했었지만, 잘 못 언급한 것 같네요.
생각해보니 recent 같은 경우 새롭게 접속한 유저 or 새로고침한 유저만 수행하는 API이고, 0-1분, 30-31분 내에 발생하는 좋아요 수 및 댓글에 대해서는 큰 문제가 없을 것이기에 무성님 PR에 동의합니다.
또한 SSE와 관련된 데이터를 저장하는 Redis 데이터에 대해서도 동기화가 되어야 하나 싶었는데, SSE와 관련된 데이터에서는 게시글 ID만 얻고 이를 토대로 DB에서 조회를 하기에 동기화가 필요없다는 것을 느꼈습니다! 작업 고민하시느라 고생 많으셨습니다.

@CodeVac513 CodeVac513 merged commit 142b0c8 into main Aug 28, 2025
2 checks passed
@CodeVac513 CodeVac513 deleted the feat/add-comment-count-to-redis branch August 28, 2025 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BE] 게시글 정보 업데이트시 Redis 데이터 동기화

4 participants