[HSC-424] 로그 feature dispatch 날짜 및 enum 매핑 보완

[tkv00]

📝작업 내용

👀변경 사항

🎫 Jira Ticket

• Jira Ticket: HSC-424

#️⃣관련 이슈

• closes [HSC-424] user-log admin dispatch 아웃박스 적재 경량화 및 재시도 완화 #281

[github-actions]

🧪 Test Coverage Report (JaCoCo)

❗ JaCoCo XML not found: ./build/reports/jacoco/test/jacocoTestReport.xml

• Ensure Gradle args include: jacocoTestReport
• Expected XML: build/reports/jacoco/test/jacocoTestReport.xml

[gemini-code-assist]

Code Review

이번 PR은 로그 처리 로직의 시간대 상수를 공통화하고 Outbox 저장 로직을 별도 서비스로 분리하는 리팩토링을 수행합니다. 리뷰 결과, REQUIRES_NEW 전파 속성 사용에 따른 트랜잭션 원자성 결여, 트랜잭션 내 예외 처리 미흡으로 인한 UnexpectedRollbackException 발생 가능성, 그리고 배치 처리 시 개별 항목에 대한 예외 처리 누락 등의 문제가 발견되어 수정이 필요합니다.

[gemini-code-assist]

🛡️ Code Review Report

1. 🔍 요약

Outbox 패턴의 트랜잭션 원자성 보장 및 예외 처리 방식에 대한 개선이 필요합니다. 특히 REQUIRES_NEW 사용으로 인한 데이터 불일치 가능성과 트랜잭션 내 예외 처리 시 발생하는 UnexpectedRollbackException 문제를 해결해야 합니다.

2. 🛑 Blocking Issues (Must Fix)

• [위반 규칙]: Application Layer Transaction Boundaries (Style Guide Line 43)
• [문제 이유]: Propagation.REQUIRES_NEW를 사용하면 비즈니스 트랜잭션이 롤백되더라도 로그 저장 트랜잭션은 독립적으로 커밋됩니다. 이는 실제 수행되지 않은 작업에 대한 로그가 Admin 서비스로 전송되는 결과를 초래하여 데이터 정합성을 깨뜨립니다.
• [해결 제안]: 기본 전파 속성인 REQUIRED를 사용하여 비즈니스 트랜잭션과 원자성을 유지하도록 수정하세요.

Suggested change

	@Transactional(propagation = Propagation.REQUIRES_NEW)
	@Transactional

References

1. Application Layer (UseCase) handles transaction boundaries. @transactional is allowed here. ^(link)

[gemini-code-assist]

🛑 Blocking Issue: 트랜잭션 내 예외 처리 및 UnexpectedRollbackException

• [문제 이유]: @Transactional 메서드 내부에서 런타임 예외(DataIntegrityViolationException)를 catch하고 다시 던지지 않으면, 트랜잭션은 이미 rollback-only로 마킹된 상태에서 커밋을 시도하게 되어 UnexpectedRollbackException이 발생합니다. 이로 인해 호출부의 fallback 로직(forEach)이 첫 번째 실패 건에서 중단되어 나머지 데이터 처리가 불가능해집니다.
• [해결 제안]: store 메서드 내부의 try-catch를 제거하고, 예외 처리를 트랜잭션 경계 밖인 UserLogAdminDispatchOutboxService로 이동시키세요.

    @Transactional
    public void store(UserLogAdminDispatchOutbox row) {
        repository.saveAndFlush(row);
        customerMetrics.recordAdminLogFeatureOutbox("stored");
    }

[gemini-code-assist]

🛑 Blocking Issue: Fallback 로직의 예외 전파 차단

• [문제 이유]: stateService.store(row) 호출 시 발생하는 예외(특히 UnexpectedRollbackException)가 forEach 루프 내부에서 처리되지 않습니다. 중복 데이터 등으로 인해 한 건이라도 실패하면 전체 루프가 중단되고 나머지 로그들이 저장되지 못합니다.
• [해결 제안]: 루프 내부에서 개별적으로 예외를 처리하여 실패한 건을 제외한 나머지 데이터는 정상적으로 저장될 수 있도록 보장해야 합니다.

            rows.forEach(row -> {
                try {
                    stateService.store(row);
                } catch (Exception e) {
                    log.warn("[UserLog][Outbox] individual store failed. event_id={}", row.getEventId());
                }
            });