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

[Refactor]: 커뮤니티 게시글 조회수 비관적 락 적용 #571

Merged
merged 2 commits into from
Jan 7, 2025

Conversation

dev-Crayon
Copy link
Contributor

🐬 요약

기존에 적용되어 있던 Bucket이 모든 유저가 하나의 버킷을 공유하고 있는 것을 확인
버킷 관련 로직을 삭제하고 비관적 락을 적용해서 데이터 정합성 확보 및 게시글 조회와 동시 테스트를 통해 낙관적 락보다 비관적 락이
더 적합하다는 것을 파악

👻 유형

PR의 유형에 맞게 체크해주세요!

  • 버그 수정
  • 기능 개발
  • 코드 스타일 수정 (formatting, local variables)
  • 리팩토링 (no functional changes, no api changes)
  • 빌드 관련 변경사항
  • CI 관련 변경사항
  • CD 관련 변경사항
  • 문서 내용 변경
  • Release
  • 기타... (다음 줄에 사유를 입력해주세요)

🍀 작업 내용

PR에 담긴 작업 내용을 작성해주세요!

  • 버킷 로직 삭제
  • 비관적락 적용

🌟 관련 이슈

PR과 관련된 이슈 번호를 작성해주세요!

close: #565

@dev-Crayon dev-Crayon added high priority 높은 우선순위 요청 작업 승헌✏️ 승헌이 연 PR, Issue에 사용 ⛏ Fix 버그, 오류 해결 size/XL labels Dec 29, 2024
@dev-Crayon dev-Crayon requested a review from jun02160 December 29, 2024 06:51
@dev-Crayon dev-Crayon self-assigned this Dec 29, 2024
Copy link

height bot commented Dec 29, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Member

@jun02160 jun02160 left a comment

Choose a reason for hiding this comment

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

버킷을 사용하지 않게 됨으로써 조회 수 증가 쿼리에서 잡는 락이 다른 API에도 영향을 줄 수 있을 것 같은데 요 부분은 고려가 되었을까요??

@dev-Crayon
Copy link
Contributor Author

로컬 환경에서 테스트를 진행했을 때 게시글 조회를 동시에 테스트했는데 평균 600ms 정도의 응답시간을 보였습니다.
테스트 환경은 1000개의 가상 스레드로 게시물 조회수 증가 API를 요청하고 1000개의 가상 스레드로 게시물 조회를 했을 때의 결과라 실제 저희 프로젝트에서 발생하는 트래픽보다 훨씬 많은 상황을 가정했다고 생각합니다
크게 문제 없을 것 같아요!

@jun02160
Copy link
Member

jun02160 commented Jan 5, 2025

@dev-Crayon 넵 너무 좋네요. 머지해도 문제 없을 것 같습니다!! 수고 많으셨습니다 🚀🚀

@dev-Crayon dev-Crayon merged commit af5f157 into develop Jan 7, 2025
1 check passed
@dev-Crayon dev-Crayon deleted the refactor/#565-viewCount branch January 7, 2025 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛏ Fix 버그, 오류 해결 high priority 높은 우선순위 요청 작업 size/M 승헌✏️ 승헌이 연 PR, Issue에 사용
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] 커뮤니티 조회수 API 리팩토링
2 participants