[TEST] 시연용 테스트 코드#381
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the cron execution times for scheduled tasks in AlarmSchedulerService and PinGeoScheduler. The feedback suggests externalizing these hardcoded cron expressions using Spring property placeholders (${...}) to allow flexible configuration across different environments.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| // 기존 설정 시각 | ||
| //@Scheduled(cron = "0 0 13 * * *", zone = "Asia/Seoul") | ||
| @Scheduled(cron = "0 0 14 * * *", zone = "Asia/Seoul") | ||
| @Scheduled(cron = "0 8 15 * * *", zone = "Asia/Seoul") |
There was a problem hiding this comment.
스케줄러의 cron 표현식이 소스 코드에 하드코딩되어 있습니다. 시연 및 테스트 환경이나 운영 환경에 따라 스케줄러 주기를 유연하게 변경할 수 있도록, Spring의 프로퍼티 플레이스홀더(${...})를 사용하여 외부 설정 파일(application.yml 등)에서 관리할 수 있도록 개선하는 것을 권장합니다. 기본값을 지정해 두면 별도의 설정이 없을 때도 안전하게 동작합니다.
| @Scheduled(cron = "0 8 15 * * *", zone = "Asia/Seoul") | |
| @Scheduled(cron = "${schedules.alarm.event:0 8 15 * * *}", zone = "Asia/Seoul") |
| // 기존 설정 시각 | ||
| //@Scheduled(cron = "0 0 10 * * *", zone = "Asia/Seoul") | ||
| @Scheduled(cron = "1 0 14 * * *", zone = "Asia/Seoul") | ||
| @Scheduled(cron = "1 8 15 * * *", zone = "Asia/Seoul") |
There was a problem hiding this comment.
상점 알림 스케줄러의 cron 표현식도 외부 설정을 통해 관리할 수 있도록 플레이스홀더 형식으로 개선하는 것을 권장합니다.
| @Scheduled(cron = "1 8 15 * * *", zone = "Asia/Seoul") | |
| @Scheduled(cron = "${schedules.alarm.store:1 8 15 * * *}", zone = "Asia/Seoul") |
| // 기존 설정 시각 | ||
| //@Scheduled(cron = "0 0 18 * * *", zone = "Asia/Seoul") | ||
| @Scheduled(cron = "2 0 14 * * *", zone = "Asia/Seoul") | ||
| @Scheduled(cron = "2 8 15 * * *", zone = "Asia/Seoul") |
There was a problem hiding this comment.
인기 알림 스케줄러의 cron 표현식도 외부 설정을 통해 관리할 수 있도록 플레이스홀더 형식으로 개선하는 것을 권장합니다.
| @Scheduled(cron = "2 8 15 * * *", zone = "Asia/Seoul") | |
| @Scheduled(cron = "${schedules.alarm.hot:2 8 15 * * *}", zone = "Asia/Seoul") |
| private final RedisTemplate<String, String> redisTemplate; | ||
|
|
||
| @Scheduled(cron = "0 25 13 * * *", zone = "Asia/Seoul") | ||
| @Scheduled(cron = "0 0 5 * * *", zone = "Asia/Seoul") |
There was a problem hiding this comment.
캐시 재구성 스케줄러의 cron 표현식도 외부 설정을 통해 관리할 수 있도록 플레이스홀더 형식으로 개선하는 것을 권장합니다.
| @Scheduled(cron = "0 0 5 * * *", zone = "Asia/Seoul") | |
| @Scheduled(cron = "${schedules.geo.rebuild:0 0 5 * * *}", zone = "Asia/Seoul") |
1 participant
🔗 Related Issue
✨ 작업 개요
체크리스트
📷 이미지 첨부 (선택)
🧐 집중 리뷰 요청