Conversation
[HSC-65] feat: 상품 조회 Web 계층 (api 응답 dto, mapper, controller)
[HCR-64] feat: 회원 & 상품 관련 스키마 설계 및 JPA 엔티티 매핑
[HSC-423] admin log-feature dispatch rejection 정책 변경 및 reject 계측 추가
[HSC-424] user-log admin dispatch outbox 저장 및 재처리 경로 구성
[HSC-424] user-log admin dispatch 아웃박스 적재 경량화 및 재시도 완화
[HSC-424] 로그 feature dispatch 날짜 및 enum 매핑 보완
[HSC-424] 로그 outbox 서비스 컴파일 오류 정리
[HSC-424] 로그 feature 배치 엔드포인트 추가
[HCR-425] Hotfix/hsc 425
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 PR은 지역별 통계 조회를 위한 Caffeine 캐시 도입과 사용자 로그의 안정적인 전송을 위한 아웃박스(Outbox) 패턴 및 배치 처리 로직을 구현합니다. AdminRegionalTopPlanDao의 jOOQ 쿼리 최적화와 추천 로직의 UseCase 분리가 포함되었습니다. 리뷰 피드백으로는 관리자 API 엔드포인트 명명 규칙 준수, 하드코딩된 타임존 상수의 shared 패키지 공통화, 카프카 컨슈머의 과도한 로그 레벨 변경, 그리고 application.yaml에서 의도치 않게 주석 처리된 로깅 설정 복구를 요청했습니다.
| return ResponseEntity.accepted().build(); | ||
| } | ||
|
|
||
| @PostMapping("/batch") |
There was a problem hiding this comment.
프로젝트 아키텍처 규칙(Rule 36) 및 일반 규칙에 따라 Admin API 엔드포인트는 반드시 /api/admin/... 경로를 포함해야 하며, 버전 정보가 앞에 와야 합니다 (/api/{version}/admin/...). 현재 클래스의 베이스 경로(/internal/v1/log-features)가 이 규칙을 위반하고 있으므로 수정이 필요합니다.
References
- Admin 엔드포인트는 @Profile("admin")과 @RequestMapping("/api/admin/...")을 가져야 함 (link)
| @RequiredArgsConstructor | ||
| public class UserLogAdminDispatchOutboxService { | ||
|
|
||
| private static final ZoneId BASE_DATE_ZONE = ZoneId.of("Asia/Seoul"); |
There was a problem hiding this comment.
시간대 설정(Asia/Seoul)이 HandleLogFeatureUseCase 등 여러 클래스에서 중복되어 하드코딩되어 있습니다. 아키텍처 규칙(Rule 27)에 따라 공통 상수는 shared 패키지로 추출하여 관리하는 것이 유지보수 측면에서 바람직합니다.
References
- 공통 유틸리티는 shared 또는 shared.domain에 위치해야 함 (link)
Comment on lines
+45
to
+48
| log.info( | ||
| "[REC][trace_id={}][member_id={}] stage={} elapsed_ms={} topic={} offset={}", | ||
| traceId, "unknown", "received", elapsedMs(consumeStartedAt), topic, offset | ||
| ); |
There was a problem hiding this comment.
Kafka 컨슈머에서 메시지 처리 단계별로 info 레벨 로그를 과도하게 출력하고 있습니다. 처리량이 많은 환경에서는 성능 저하 및 로그 저장 비용 증가를 초래할 수 있습니다. 상세 추적 로그는 debug 레벨로 변경하거나, 처리가 완료된 시점에 하나의 통합된 로그를 남기는 방식을 권장합니다.
Comment on lines
+108
to
+119
| #logging: | ||
| # level: | ||
| # root: WARN | ||
| # ACCESS_LOG: INFO | ||
| # structured: | ||
| # format: | ||
| # console: logstash | ||
| # json: | ||
| # stacktrace: | ||
| # max-length: 5000 | ||
| # include-common-frames: true | ||
| # include-hashes: true |
There was a problem hiding this comment.
logging 설정이 주석 처리되어 있습니다. 이로 인해 운영 환경에서 구조화된 로그 형식이 적용되지 않거나 로그 레벨 제어가 어려워질 수 있습니다. 의도된 변경이 아니라면 주석을 해제해야 합니다.
🧪 Test Coverage Report (JaCoCo)
Package line spark (sample): 📦 Package coverage (worst 10)
🧨 Lowest coverage classes (worst 10)
🧩 Changed files coverage breakdown (24 files)
🔎 HTML 리포트: Actions → Artifacts → |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
📝작업 내용
👀변경 사항
🎫 Jira Ticket
#️⃣관련 이슈