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

#538 [feat] 모임당 인기글/ 인기 작가 로직 수정 #543

Merged
merged 4 commits into from
Oct 20, 2024

Conversation

sohyundoh
Copy link
Member

✒️ 관련 이슈번호

Key Changes 🔑

  1. 만약 인기 작가/ 글 을 연산하였을 때 null 값이면 cache에 그대로 null을 저장해 null 이 리턴되는 문제가 있었습니다. cache에 null 값이 저장되지 않게 수정하였습니다 (논의 후 결정)
  2. 그리고 인기 작가/글이 조회 때문에 발생하여 동시성 문제를 해결하기 위해 AOP 기반의 분산락을 구현했습니다.
  3. 분산락을 구현하던 도중 내부 호출 함수에 어노테이션을 걸어놓으니, 분산락이 작동하지 않아서 (프록시 객체로 사용해야하는데) 클래스를 분리했습니다.

To Reviewers 📢

@sohyundoh sohyundoh self-assigned this Oct 20, 2024
@sohyundoh sohyundoh merged commit cf006aa into develop Oct 20, 2024
1 check passed
@sohyundoh sohyundoh deleted the feat/#538 branch October 20, 2024 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 인기글 관련 로직 변경
1 participant