Skip to content

[TEST] 알람 스케쥴링 재테스트#346

Merged
taerimiiii merged 1 commit into
developfrom
test/339-alarm
Jun 15, 2026
Merged

[TEST] 알람 스케쥴링 재테스트#346
taerimiiii merged 1 commit into
developfrom
test/339-alarm

Conversation

@taerimiiii

Copy link
Copy Markdown
Contributor

🔗 Related Issue

  • resolve #이슈번호

✨ 작업 개요

작업 내용을 간략하게 작성해주세요.

체크리스트

  • Reviewers, Assignees, Labels를 모두 등록했나요?
  • .gitignore 설정을 하였나요?
  • PR 머지 전 반드시 CI가 정상적으로 작동하는지 확인해주세요!

📷 이미지 첨부 (선택)

  • 작업 결과를 확인할 수 있는 이미지나 GIF를 첨부해주세요.
  • UI 변경, API 응답 샘플, 테스트 결과 등이 포함되면 좋아요!

🧐 집중 리뷰 요청

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

@taerimiiii taerimiiii self-assigned this Jun 15, 2026
@taerimiiii taerimiiii added the 🧪 test 테스트 코드 label Jun 15, 2026
@taerimiiii taerimiiii merged commit 92df0e5 into develop Jun 15, 2026
1 check passed

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the scheduled execution times for three alarm services (sendEventAlarms, sendStoreAlarms, and sendHotAlarms) to run at 22:15. The reviewer noted that scheduling all three tasks at the exact same time can cause sequential execution delays under Spring's default single-threaded scheduler, and suggested staggering the times or configuring a thread pool.

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.

Comment on lines +20 to +34
@Scheduled(cron = "0 15 22 * * *", zone = "Asia/Seoul")
public void sendEventAlarms() {
regionalAlarmCommandService.dispatchScheduledEventAlarms();
}

// 기존 설정 시각
//@Scheduled(cron = "0 0 10 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 45 21 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 15 22 * * *", zone = "Asia/Seoul")
public void sendStoreAlarms() {
regionalAlarmCommandService.dispatchScheduledStoreAlarms();
}

// 기존 설정 시각
//@Scheduled(cron = "0 0 18 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 45 21 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 15 22 * * *", zone = "Asia/Seoul")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

현재 세 개의 스케줄러 작업(sendEventAlarms, sendStoreAlarms, sendHotAlarms)이 모두 동일한 시간(22:15:00)에 실행되도록 설정되어 있습니다. Spring의 기본 TaskScheduler는 단일 스레드로 동작하므로, 이 작업들이 병렬로 실행되지 않고 순차적으로 실행되어 이전 작업의 실행 시간에 따라 다음 작업이 지연될 수 있습니다. 작업 간의 실행 시간을 몇 분 간격으로 분산(staggering)시키거나, SchedulerConfig에서 스레드 풀을 설정하여 병렬 처리가 가능하도록 개선하는 것을 권장합니다.

Suggested change
@Scheduled(cron = "0 15 22 * * *", zone = "Asia/Seoul")
public void sendEventAlarms() {
regionalAlarmCommandService.dispatchScheduledEventAlarms();
}
// 기존 설정 시각
//@Scheduled(cron = "0 0 10 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 45 21 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 15 22 * * *", zone = "Asia/Seoul")
public void sendStoreAlarms() {
regionalAlarmCommandService.dispatchScheduledStoreAlarms();
}
// 기존 설정 시각
//@Scheduled(cron = "0 0 18 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 45 21 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 15 22 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 15 22 * * *", zone = "Asia/Seoul")
public void sendEventAlarms() {
regionalAlarmCommandService.dispatchScheduledEventAlarms();
}
// 기존 설정 시각
//@Scheduled(cron = "0 0 10 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 16 22 * * *", zone = "Asia/Seoul")
public void sendStoreAlarms() {
regionalAlarmCommandService.dispatchScheduledStoreAlarms();
}
// 기존 설정 시각
//@Scheduled(cron = "0 0 18 * * *", zone = "Asia/Seoul")
@Scheduled(cron = "0 17 22 * * *", zone = "Asia/Seoul")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧪 test 테스트 코드

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant