Skip to content

[BE-#625] 에러 로그 추가#626

Merged
yejun05011 merged 1 commit intomainfrom
refactor/BE-#625/refactor/ranking-logging
Feb 22, 2026
Merged

[BE-#625] 에러 로그 추가#626
yejun05011 merged 1 commit intomainfrom
refactor/BE-#625/refactor/ranking-logging

Conversation

@yejun05011
Copy link
Collaborator

PR 종류

  • 리팩토링

변경 사항

  • 랭킹 도메인 전반에 로깅 정책 적용
  • RankingCheckInApplicationService 예외 처리 및 ERROR 로그 추가
  • RankingEventTriggerService 이벤트 발행 실패 로깅 추가
  • KafkaRankingEventPublisher 비동기 전송 실패 로깅 보강
  • RedisRankingStore 주요 연산(increaseScore, getRank, getAllRankings) 예외 로깅 추가
  • SnapshotJob / SnapshotService DB 저장 실패 시 ERROR 로그 및 throw 전파
  • RankingScheduler 스케줄 시작/성공/실패 로그 추가

관련 이슈

체크리스트

  • 테스트 코드를 작성하였나요? (기존 테스트 영향 없음 확인)
  • 모든 테스트가 통과하나요?
  • 코드 컨벤션을 지켰나요?
  • 관련 문서를 업데이트했나요?

상세 내용

상황 로그 레벨
점수 정상 반영 INFO
랭킹 변동 감지 INFO
Kafka 이벤트 발행 실패 ERROR
Redis 연산 실패 ERROR
Snapshot 저장 실패 ERROR
내부 로직 오류 ERROR
  • Application 레이어: 비즈니스 성공을 방해하지 않도록 예외 로깅 후 삼킴
  • Infrastructure 레이어: 인프라 오류는 ERROR 로그 후 throw
  • Snapshot/스케줄러: 데이터 무결성 보장을 위해 ERROR 로그 후 throw

@yejun05011 yejun05011 self-assigned this Feb 21, 2026
@yejun05011 yejun05011 added the refactor 기능 리팩토링 label Feb 21, 2026
Copy link
Collaborator

@Dayoung0402 Dayoung0402 left a comment

Choose a reason for hiding this comment

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

고생하셨어요! 머지하셔도 될거 같습니다.

@yejun05011 yejun05011 merged commit 7eeb57b into main Feb 22, 2026
1 check passed
@yejun05011 yejun05011 linked an issue Feb 26, 2026 that may be closed by this pull request
@yejun05011 yejun05011 deleted the refactor/BE-#625/refactor/ranking-logging branch February 26, 2026 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 기능 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] 에러 로그 추가

2 participants