Skip to content

fix: Read Model 누락 시 도메인 이벤트 재시도 처리#205

Merged
lunarbae628 merged 2 commits into
stagingfrom
fix/read-model-event-retry
Jun 5, 2026
Merged

fix: Read Model 누락 시 도메인 이벤트 재시도 처리#205
lunarbae628 merged 2 commits into
stagingfrom
fix/read-model-event-retry

Conversation

@lunarbae628
Copy link
Copy Markdown
Collaborator

@lunarbae628 lunarbae628 commented Jun 5, 2026

🛰️ Issue Number

  • 미지정

🪐 작업 내용

  • DocListProjector가 변경/삭제 이벤트 처리 중 대상 DocListReadModel을 찾지 못하면 IllegalStateException을 던지도록 변경했습니다.
  • 기존에는 read model 누락 시 이벤트를 조용히 무시할 수 있었고, 이 경우 outbox가 DONE 처리되어 이후 재처리 기회가 사라질 수 있었습니다.
  • 누락 상황을 예외로 남겨 DomainEventOutboxRelay의 실패/재시도 경로를 타도록 했습니다.
  • read model 생성 후 기존 변경 이벤트가 재시도되어 반영되는 통합 테스트와 각 이벤트 타입별 누락 예외 단위 테스트를 추가했습니다.
  • 관련 AI 작업 흐름 문서, 설계/계획 문서, 읽기 성능 벤치마크 스크립트가 함께 포함되어 있습니다.

📚 Reference

  • 없음

🤖 AI 보조 작업 기록

  • 사용 범위: PR 생성 요청에 따라 원격 브랜치의 변경 범위, 기준 브랜치, 중복 PR 여부를 확인하고 PR 제목/본문을 작성했습니다.
  • 채택한 제안과 이유: dev를 기준 브랜치로 사용했습니다. 저장소 기본 브랜치가 dev로 확인되었습니다.
  • 기각한 제안과 이유: 없음
  • 실행한 검증: git fetch origin, git log origin/dev..origin/fix/read-model-event-retry, git diff --stat origin/dev...origin/fix/read-model-event-retry, 중복 PR 검색
  • 남은 리스크: 현재 로컬 checkout은 다른 브랜치이며 미커밋 변경이 있어, 이 PR 생성 단계에서는 테스트/빌드를 새로 실행하지 않았습니다.

✅ Check List

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

@lunarbae628 lunarbae628 self-assigned this Jun 5, 2026
@lunarbae628
Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🎉

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lunarbae628 lunarbae628 changed the base branch from dev to staging June 5, 2026 17:36
@lunarbae628 lunarbae628 marked this pull request as ready for review June 5, 2026 17:38
@lunarbae628 lunarbae628 merged commit 99779ed into staging Jun 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant