Skip to content

fix: 캐시 주석 해제#705

Merged
JAEHEE25 merged 1 commit intodevelopfrom
fix/704-redis-cache
Mar 24, 2026
Merged

fix: 캐시 주석 해제#705
JAEHEE25 merged 1 commit intodevelopfrom
fix/704-redis-cache

Conversation

@JAEHEE25
Copy link
Contributor

관련 이슈

작업 내용

테스트를 위한 주석 해제

특이 사항

리뷰 요구사항 (선택)

@coderabbitai
Copy link

coderabbitai bot commented Mar 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 312ce0b6-ae7c-4fc2-8b65-d9ece740cbf8

📥 Commits

Reviewing files that changed from the base of the PR and between 48821f4 and 0ce03a2.

📒 Files selected for processing (1)
  • src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java

Walkthrough

UnivApplyInfoQueryServicesearchUnivApplyInfoByText(String text) 메서드에 캐싱 기능이 추가되었습니다. 다음과 같이 구성됩니다:

  1. 캐싱 어노테이션 적용

    • @ThunderingHerdCaching 어노테이션을 메서드에 활성화
    • 캐시 키 패턴: "univApplyInfoTextSearch:{0}"
    • 캐시 관리자: "customCacheManager"
    • TTL(Time To Live): 86400초(1일)
  2. 메서드 동작 유지

    • @Transactional(readOnly = true) 어노테이션은 그대로 유지
    • 메서드의 공개 시그니처와 타입에는 변경 사항 없음
    • 내부 쿼리 및 제어 흐름은 유지됨

이제 메서드 호출 시 결과가 지정된 캐시에 저장되어 동일한 검색에 대한 반복 호출 시 성능이 향상될 것입니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/704-redis-cache

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@JAEHEE25 JAEHEE25 merged commit 3fae55e into develop Mar 24, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: redis 캐시 오류 수정

1 participant