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

[Feature] - Redis cache 도입 #629

Merged
merged 8 commits into from
Dec 31, 2024
Merged

[Feature] - Redis cache 도입 #629

merged 8 commits into from
Dec 31, 2024

Conversation

eunjungL
Copy link

✅ 작업 내용

  • application.yml에 Redis 연결 정보 추가
  • Redis Config 작성
  • CacheManager Redis로 변경
  • Testcontainers Redis 추가

🙈 참고 사항

S3처럼 local에서 EmbeddedRedis를 사용할까 했는데 다음을 이유로 그러지 않기로 했습니다.

  • 공식 라이브러리가 아닌 개인 소유의 라이브러리를 사용해야 함 (신뢰성 낮음)
  • 라이브러리 노후화 (최신 업데이트가 6년전 ㄷㄷ redis 2.X 버전까지만 기능 업데이트)
  • 불친절한 예외 메시지등 낮은 사용성

우선은 local 프로파일로 서버를 돌려보고 싶으시다면 redis를 실행하고 사용하시면 됩니다~
테스트의 경우에는Testcontainers를 통한 통합테스트 추상 클래스에 redis 컨테이너를 추가했으니 신경쓰지 않으셔도 됩니다.

@eunjungL eunjungL added this to the sprint 8 milestone Dec 31, 2024
@eunjungL eunjungL linked an issue Dec 31, 2024 that may be closed by this pull request
4 tasks
Copy link

Test Results

 30 files   30 suites   1m 1s ⏱️
296 tests 295 ✅ 1 💤 0 ❌
308 runs  307 ✅ 1 💤 0 ❌

Results for commit 41ae2d5.

Copy link

@slimsha2dy slimsha2dy left a comment

Choose a reason for hiding this comment

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

레디스 캐시 설정 및 테스트 컨테이너 설정하신 것 확인했습니다!
바로 어프로브하겠습니다~

@eunjungL eunjungL merged commit 4006813 into develop/be Dec 31, 2024
3 checks passed
@eunjungL eunjungL deleted the feature/be/#628 branch December 31, 2024 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Feature] - Redis cache 도입
3 participants