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

좋아요 기능 버그 해결을 위한 캐싱 자료구조 및 로직 변경 #791

Merged
merged 29 commits into from
Feb 5, 2024
Merged

좋아요 기능 버그 해결을 위한 캐싱 자료구조 및 로직 변경 #791

merged 29 commits into from
Feb 5, 2024

Conversation

mcodnjs
Copy link
Collaborator

@mcodnjs mcodnjs commented Feb 5, 2024

📄 Summary

#784, #788

🙋🏻 More

@mcodnjs
fix: LikeCount 업데이트 로직 수정
c033764
@mcodnjs
refactor: updateMemberLikeCache 메서드 분리
52b1d06
@mcodnjs
fix: 커뮤니티 여행 전체 조회 시 Redis 사용하도록 수정
ccbc23b
@mcodnjs
feat: RedisTemplate 빈 등록
968ca26
@mcodnjs
feat: 좋아요 업데이트 로직 구현
5689976
@mcodnjs
feat: db와 redis 동기화 스케줄러 구현
12d48c8
@mcodnjs
feat: 좋아요 조회 로직 구현
5373b30
@mcodnjs
refactor: 사용하지 않는 메서드 삭제
23378f7
@mcodnjs
refactor: LikeElement 필드 변경
7109d2d
@mcodnjs
refactor: Likes 테이블에 없는 tripId에 default 값 할당
d1c4364
@mcodnjs
refactor: LikeRedisKeyConstants 생성
418b3e6
@mcodnjs
refactor: 사용하지 않는 dto 삭제
64dbb6f
@mcodnjs
refactor: like ttl 상수화
90984b3
@mcodnjs
fix: likes 테이블 소문자로 변경
ee61595
@mcodnjs
fix: 커스텀 쿼리로 변경
79335e8
@mcodnjs
fix: 캐시된 tripId가 하나라도 있을 경우만 db 조회하도록 변경
56bb511
@mcodnjs
fix: likes 조회 시 memberIds 파싱 로직 수정
7043b79
@mcodnjs
fix: redis에 가변인자로 추가하도록 변경
2ea5b03
@mcodnjs
fix: like key prefix 수정
4583c0e
@mcodnjs
fix: like key prefix 수정
68882fe
@mcodnjs
fix: empty_marker 타입 변경
2378fa4
@mcodnjs
fix: 업데이트 시 캐시가 없는 경우 DB에서 조회해오도록 수정
7c2e8ff
@mcodnjs
refactor: 업데이트 메서드 인자 수정 및 로직 리팩토링
d33ccad
@mcodnjs
refactor: toLikeInfo로 메서드 네이밍 변경
59edb00
@mcodnjs
refactor: likeElements로 변수명 변경
c658576
@mcodnjs
refactor: 메서드 위치 변경
4a62d4c
@mcodnjs
refactor: 메서드명 변경 및 LikeInfo dto 패키지로 이동
babb205
@mcodnjs
test: LikeService 테스트 추가
8530e26
@mcodnjs
test: LikeSyncScheduler 테스트 추가
339be7b
@mcodnjs mcodnjs added the BE label Feb 5, 2024
@mcodnjs mcodnjs self-assigned this Feb 5, 2024
@mcodnjs mcodnjs linked an issue Feb 5, 2024 that may be closed by this pull request
커뮤니티 여행 좋아요 버그 수정 #784
Open
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 5, 2024

📝 Jacoco Test Coverage

Total Project Coverage 80.37% 🍏
File Coverage [87.28%]
LikeInfo.java 100% 🍏
LikeElement.java 100% 🍏
RedisConfig.java 100% 🍏
LikeSyncScheduler.java 99.07% 🍏
LikeService.java 94.66% 🍏
LikeElements.java 92.86% 🍏
CommunityService.java 85.19% 🍏
LikeRedisConstants.java 76.92% ❌
CustomLikeRepositoryImpl.java 67.46% ❌

@mcodnjs mcodnjs merged commit b9f5e8e into main Feb 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mcodnjs mcodnjs

Labels
BE 🐞 Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

커뮤니티 여행 좋아요 버그 수정
1 participant
@mcodnjs