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

[REFACTOR] 멘션 사용자 조회 API 캐시 도입 #244

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from
Open

[REFACTOR] 멘션 사용자 조회 API 캐시 도입 #244

wants to merge 14 commits into from

Conversation

mikekks
Copy link
Member

@mikekks mikekks commented Jul 9, 2024 •
edited
Loading

👩‍💻 Contents

  • 멘션 사용자 조회 API에 캐시 도입했습니다.
  • 한 기수동안 수정이 거의 일어나지 않기 때문에 캐시 히트율이 매우 높다고 생각했습니다.
  • 하지만 성능 이슈가 발생하면 도입할 예정이고, 일단 PR만 올려보고자 했습니다!
  • 캐시는 5시간마다 업데이트하도록 일단 설정해뒀고, 이후에는 새 회원이 등록될 때마다 캐시를 업데이트 하는 방식으로 변경하고자 합니다!
  • 아래 성능을 보면 평균응답시간이 확실히 빨라진 것을 확인할 수 있습니다!
스크린샷 2024-07-09 오후 1 52 45 스크린샷 2024-07-09 오후 1 53 17

📝 Review Note

📣 Related Issue

✅ 점검사항

  • docker-compose.yml 파일에 마이그레이션 한 API의 포워딩을 변경해줬나요?
  • Spring Secret 값을 수정하거나 추가했다면 Github Secret에서 수정을 해줬나요?
  • Nestjs Secret 값을 수정하거나 추가했다면 Docker-Compose.yml 파일 및 인스턴스 내부의 .env 파일을 수정했나요?

@mikekks mikekks requested a review from sgh002400 July 9, 2024 04:54
@mikekks mikekks self-assigned this Jul 9, 2024
@height Height
Copy link

height bot commented Jul 9, 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.

sgh002400
sgh002400 approved these changes Jul 9, 2024
View reviewed changes
Copy link
Contributor

@sgh002400 sgh002400 left a comment

Choose a reason for hiding this comment

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

레디스 설정부터 성능 측정까지 멋있어요 :) 고생하셨어요~!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sgh002400 sgh002400 sgh002400 approved these changes

Assignees

@mikekks mikekks

Labels
size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 자주 조회되는 특정 데이터 캐시 적용
2 participants
@mikekks @sgh002400