Skip to content

fix: RedisRepository 스캔 규칙 수정 #2040

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
Merged
Soundbar91 merged 5 commits into from
Oct 15, 2025
Merged

fix: RedisRepository 스캔 규칙 수정 #2040

Soundbar91 merged 5 commits into from
Oct 15, 2025

Conversation

@Soundbar91
Copy link
Collaborator

@Soundbar91 Soundbar91 commented Oct 13, 2025

🔍 개요

🚀 주요 변경 내용

  • RedisRepository 스캔 규칙을 수정했습니다.
    • Annotation -> Regex
    • 정규식은 *RedisRepository으로 설정했습니다.
  • RedisRepository 스캔 범위를 수정했습니다.
    • domain, admin -> 프로젝트 루트
    • 내부 로직 확인 결과 basePackages 에 설정된 각 경로를 스캔할 때마다 준비 과정이 있으며, 패키지를 2개로 설정하면 동일한 준비 과정이 두 번 발생
    • 해당 PR 작업 이후에서도 Redis 스캔 시간이 증가함을 확인
      • 2811ms -> 3322ms

💬 참고 사항

  • 로컬 환경에서는 해당 작업에 대한 개선 여부를 수치적으로 확인하기 어려워, 배포 이후에 결과를 확인할 예정입니다.

✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@github-actions github-actions bot added 공통 백엔드 공통으로 작업할 이슈입니다. 버그 정상적으로 동작하지 않는 문제상황입니다. labels Oct 13, 2025
@Soundbar91 Soundbar91 self-assigned this Oct 13, 2025
@github-actions
Copy link

github-actions bot commented Oct 13, 2025 •
edited
Loading

Unit Test Results

668 tests   667 ✔️  1m 13s ⏱️
164 suites      1 💤
164 files        0 ❌

Results for commit a9d9847.

♻️ This comment has been updated with latest results.

BaeJinho4028
BaeJinho4028 approved these changes Oct 15, 2025
View reviewed changes
Copy link
Collaborator

@BaeJinho4028 BaeJinho4028 left a comment

Choose a reason for hiding this comment

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

성공적이라면 다른 DB도 이렇게 네이밍 바꾸면 좋을 것 같네요 👍

@Soundbar91 Soundbar91 merged commit 4de2145 into develop Oct 15, 2025
5 checks passed
@Soundbar91 Soundbar91 deleted the fix/2039-redis-repository-scan-type branch October 15, 2025 05:50
Soundbar91 added a commit that referenced this pull request Oct 15, 2025
@Soundbar91
Revert "fix: RedisRepository 스캔 규칙 수정 (#2040)"
84accda 
This reverts commit 4de2145.
Soundbar91 added a commit that referenced this pull request Oct 15, 2025
@Soundbar91
fix: Spring Data 스캔 작업 롤백 (#2043)
0678502 
* Revert "fix: RedisRepository 스캔 규칙 수정 (#2040)"

This reverts commit 4de2145.

* Revert "fix: RedisRepository 스캔 범위 수정 (#2035)"

This reverts commit 712995b.

* Revert "refactor: Spring Data 스캔 개선 (#2026)"

This reverts commit 998849e.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BaeJinho4028 BaeJinho4028 BaeJinho4028 approved these changes

@kih1015 kih1015 kih1015 approved these changes

@ImTotem ImTotem Awaiting requested review from ImTotem

Assignees

@Soundbar91 Soundbar91

Labels

공통 백엔드 공통으로 작업할 이슈입니다. 버그 정상적으로 동작하지 않는 문제상황입니다.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[공통] RedisRepository 스캔 규칙을 정규식으로 수정한다.

4 participants

@Soundbar91 @BaeJinho4028 @kih1015