chore: master hotfix를 develop에 sync

[whqtker]

• hotfix/260218, hotfix/260220, hotfix/260221에서 적용된 변경사항을 develop에 반영

[coderabbitai]

Walkthrough

이 변경사항은 Application 저장소의 조회 메서드를 개선하고 캐싱 설정을 조정하는 작업입니다. 다음과 같이 진행됩니다:

1. Application 저장소 메서드 교체

   • 기존의 findBySiteUserIdAndTermId 메서드를 삭제된 항목을 제외하고 최신 레코드를 조회하는 findTopBySiteUserIdAndTermIdAndIsDeleteFalseOrderByIdDesc로 교체하였습니다.

2. 서비스 레이어 업데이트

   • ApplicationSubmissionService에서 새로운 저장소 메서드를 사용하도록 변경하여 삭제되지 않은 최신 신청 정보를 조회합니다.

3. 캐싱 비활성화

   • UnivApplyInfoQueryService의 searchUnivApplyInfoByText 메서드에서 ThunderingHerdCaching 어노테이션을 주석 처리하여 캐싱을 임시로 비활성화했습니다.

4. 테스트 코드 조정

   • ApplicationSubmissionServiceTest에서 새로운 저장소 메서드명으로 업데이트하고, UnivApplyInfoQueryServiceTest의 캐싱 관련 테스트 2개를 비활성화했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

• wibaek
• lsy1307
• Gyuhyeok99
• Hexeong

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	PR 설명이 매우 간략하며, 리포지토리의 필수 템플릿 섹션 대부분이 누락되었습니다.	관련 이슈 번호, 상세한 작업 내용, 특이 사항, 리뷰 요구사항 등을 포함하여 템플릿에 맞게 작성해 주세요.
Docstring Coverage	⚠️ Warning	Docstring coverage is 44.44% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 master의 hotfix를 develop에 동기화하는 변경사항의 주요 내용을 명확하게 요약하고 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch sync/master-to-develop

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

🧹 Nitpick comments (1)

src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java (1)

45-46: 캐싱 임시 비활성화 - Hotfix로 적절합니다.

1. 변경 내용:

   • searchUnivApplyInfoByText 메서드의 @ThunderingHerdCaching 어노테이션이 주석 처리됨
   • Redis 관련 에러가 원인으로 명시됨

2. 주의 사항:

   • 캐싱 비활성화로 인해 해당 API의 응답 시간이 증가할 수 있습니다
   • 트래픽이 높은 환경에서는 DB 부하를 모니터링해 주세요

3. 추적 권장:

   • 이슈 트래커에 Redis 에러 원인 분석 태스크를 등록하여 추적하시는 것을 권장드립니다

,

이슈를 생성하여 Redis 에러 원인 분석을 추적하시겠습니까?

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java`
around lines 45 - 46, The `@ThunderingHerdCaching` annotation on
searchUnivApplyInfoByText in UnivApplyInfoQueryService was temporarily commented
out due to Redis errors; create a tracking issue for investigating the Redis
root cause, then re-enable the annotation (remove the comment) once the Redis
bug/connection/config is resolved and reference the issue ID in a TODO comment
next to the restored annotation; if immediate re-enable is risky, wrap caching
behind a feature flag or conditional so the method can safely toggle caching
while you fix Redis.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java`:
- Around line 45-46: The `@ThunderingHerdCaching` annotation on
searchUnivApplyInfoByText in UnivApplyInfoQueryService was temporarily commented
out due to Redis errors; create a tracking issue for investigating the Redis
root cause, then re-enable the annotation (remove the comment) once the Redis
bug/connection/config is resolved and reference the issue ID in a TODO comment
next to the restored annotation; if immediate re-enable is risky, wrap caching
behind a feature flag or conditional so the method can safely toggle caching
while you fix Redis.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 8d803623-bca6-45d0-91b4-f1d2e765a991

📥 Commits

Reviewing files that changed from the base of the PR and between 00093eb and 8bb7429.

📒 Files selected for processing (5)

• src/main/java/com/example/solidconnection/application/repository/ApplicationRepository.java
• src/main/java/com/example/solidconnection/application/service/ApplicationSubmissionService.java
• src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java
• src/test/java/com/example/solidconnection/application/service/ApplicationSubmissionServiceTest.java
• src/test/java/com/example/solidconnection/university/service/UnivApplyInfoQueryServiceTest.java