[#29] Redis를 이용해 큐로 재시도 로직 구현 #30
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 큐를 넣었기에 관련된 테스트 메소드들을 추가했습니다.
heeve1
reviewed
May 19, 2025
|
|
||
| @Scheduled(fixedDelay = 5 * 60 * 1000) // 5분마다 실행 | ||
| public void processRetryQueue() { | ||
| while (true) { |
There was a problem hiding this comment.
현재 로직에서는 예외가 발생하면 87번 라인을 통해 실패 요청이 다시 Redis 큐에 들어가게 됩니다. 그런데 while(true) 루프 안에서 즉시 다음 요청을 꺼내 재시도하기 때문에, 같은 요청이 반복적으로 빠르게 처리되며 재시도 로직이 계속 순환하게 됩니다.
이 과정에서 재시도마다 DB 접근 등의 리소스가 동반되므로, 만약 지속적으로 실패하는 요청이 있다면 짧은 시간 안에 고빈도 DB 트래픽이 발생하고, 결국 시스템 부하나 장애로 이어질 위험이 있습니다!
따라서 무한 루프를 제한하거나, 실패한 요청에 대해 재시도 간 간격이나 최대 재시도 횟수 등을 도입하는 개선이 필요해 보이는군요. 다시 한번 검토해보시지요!
There was a problem hiding this comment.
관련 커밋 올렸습니다. 늦은 시간까지 고생이 많으시네요 쉬엄쉬엄 하셔도 됩니다ㅠ
- 포맷 수정으로 인한 관련 테스트 클래스의 객체 포맷도 수정했습니다.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #29
수정사항
SummaryGenerationService 클래스의 로직을 보충했습니다. 실패시 큐에 넣는 재시도 로직을 구현했습니다.
추후 고려사항(하지만 현재는 할 생각이 없음)