PR 테스트

[CheatIsKey]

No description provided.

[github-actions]

🤖 AI 코드리뷰

리더보드 — 현재 3위 (리뷰 96건) | 전체 8개 팀 참여 중
업데이트 확인 — 리뷰가 질문을 던지다 — 생각해보기

안녕하세요! CheatIsKey님, PR 테스트이자 대규모 프로젝트 세팅을 위한 첫 걸음을 내디디셨군요. Docker, CI/CD, K6 부하 테스트, 동시성 제어를 위한 다양한 Lock 전략까지, 실무에서 매우 중요한 인프라와 아키텍처 고민이 많이 엿보이는 훌륭한 밑바탕 코드입니다.

10년차 시니어 개발자 시점에서 코드를 꼼꼼히 리뷰하고, 과제 요구사항에 맞게 피드백을 드리겠습니다.

📋 과제 요구사항 준수 여부

• 필수 기능 (동시성 제어 / 캐싱 / 검색): 현재 PR의 diff 상으로는 핵심 비즈니스 로직(서비스 계층, 컨트롤러)이 포함되지 않아 완전한 구현 여부를 확인하기 어렵습니다. 다만, build.gradle의 설정과 EventOrderFacade 클래스를 보면 요구사항을 충족하기 위한 멋진 기반을 다져두셨습니다.
• 🔴 사용 금지 기술 사용 (Redisson): build.gradle에 redisson-spring-boot-starter:3.51.0이 추가되어 있으며, EventOrderFacade에 @RedissonLock이 사용되었습니다. 과제 요구사항상 필수 기능에서는 Redisson 사용이 금지되어 있으므로, 일반 주문 로직에서는 Lettuce 기반 Lock(@RedisLock)만 사용하시고 Redisson은 도전 기능 평가를 위해 남겨두시는 것이 좋습니다.

🟢 잘된 점

1. 체계적인 인프라 구성: Docker Compose를 활용해 MySQL, Redis, App, K6 컨테이너를 명확하게 분리하고 depends_on과 healthcheck를 설정하신 점이 훌륭합니다. 실무에서 컨테이너 기동 순서로 인해 발생하는 오류를 사전에 완벽하게 차단하셨습니다.
2. 다양한 동시성 제어 전략 구현: EventOrderFacade에서 Lettuce 기반의 Fail-Fast, Retry, Blocking 전략을 모두 구현하시고 AOP로 추상화까지 해두신 점은 아주 인상적입니다. 각 전략의 트레이드오프를 이해하고 계신 것이 잘 드러납니다.
3. 성능 테스트 대비 (K6): smoke, load, health 체크용 K6 스크립트를 미리 작성해 두어, 기능 개선 전후의 성능 지표를 정량적으로 비교할 수 있는 탄탄한 기반을 마련하셨습니다.

🔴 필수 수정

1. .env 파일 강제 로드로 인한 운영 환경 보안 위험:

   • AlldayProjectCommerceApplication의 메인 메서드에서 Dotenv를 로드하여 시스템 프로퍼티에 강제로 주입하고 있습니다.
   • 문제점: AWS(EC2, ECS 등)나 Docker 환경에서 배포할 때, 환경 변수(SPRING_PROFILES_ACTIVE, DB_URL 등)를 외부 주입 방식으로 관리해야 하는데 이 코드가 충돌을 일으키거나 의도치 않게 로컬 설정 값을 우선적으로 가져가 버릴 위험이 있습니다.
   • 해결: 로컬 개발 편의를 위한 .env 로드는 Spring Boot의 프로파일 기능을 사용하거나, 최소한 로컬 환경에서만 동작하도록 제어해야 합니다.

2. Facade 계층의 JPA 엔티티 직접 노출 (아키텍처 역전):

   • OrderPaymentFacade에서 jpa.basic.alldayprojectcommerce.domain.order.entity.Order (JPA 엔티티)를 직접 가져와 order.getUserId()로 검증하고 있습니다.
   • 문제점: Facade(Application 계층)는 도메인 계층의 엔티티 구현 기술(JPA)에 의존해서는 안 됩니다. 엔티티가 노출되면 향후 JPA가 아닌 다른 기술로 변경할 때 Facade 코드까지 수정해야 하는 유지보수 문제가 생깁니다.
   • 해결: OrderQueryService가 엔티티 대신 DTO(예: OrderResponse, OrderOwnerDto)를 반환하도록 캡슐화하세요.

🟡 권장 개선

1. 대량의 더미 데이터 JPA saveAll() 시 성능 저하 (N+1 / 대량 INSERT 이슈):

   • DummyDataProduct에서 50,000건의 데이터를 리스트에 담아 productRepository.saveAll(products)로 저장하고 있습니다.
   • 문제점: Spring Data JPA의 기본 saveAll()은 내부적으로 영속성 컨텍스트에 50,000개의 엔티티를 모두 올립니다. 1차 캐시 메모리 부족, INSERT 쿼리 5만 번 실행 등으로 인해 애플리케이션 실행 시 심각한 성능 저하 및 OutOfMemory가 발생할 수 있습니다.
   • 해결 코드 (JDBC Batch Insert 권장):

     // build.gradle 의존성 추가
     + implementation 'org.springframework.boot:spring-boot-starter-jdbc'

     // DummyDataProduct.java 일부 수정
     @Component
     @RequiredArgsConstructor
     public class DummyDataProduct implements CommandLineRunner {
     
         private final JdbcTemplate jdbcTemplate; // JdbcTemplate 주입
     
         @Override
         @Transactional
         public void run(String... args) {
             if (productRepository.count() > 0) return;
     
             String sql = "INSERT INTO product (name, price, stock, description, status, category, image_url) VALUES (?, ?, ?, ?, ?, ?, ?)";
             
             // 50,000건 Batch Insert
             jdbcTemplate.batchUpdate(sql, new BatchPreparedStatementSetter() {
                 @Override
                 public void setValues(PreparedStatement ps, int i) throws SQLException {
                     // products 리스트의 i번째 데이터를 ps에 바인딩
                 }
                 @Override
                 public int getBatchSize() {
                     return 50000;
                 }
             });
         }
     }

2. Dockerfile 멀티 스테이지 빌드 미적용:

   • 현재 eclipse-temurin:21-jdk (JDK 포함, 약 500MB 이상)를 런타임 이미지로 사용하고 계십니다.
   • 개선 제안: 빌드 환경과 실행 환경을 분리하여 eclipse-temurin:21-jre (약 200MB 대)를 사용하면 컨테이너 크기를 절반 이하로 줄일 수 있고, 공격 표면(Attack Surface)도 줄일 수 있습니다.

💡 학습 포인트

• Spring Boot 프로파일(Profile)과 외부 설정: .env 파일을 하드코딩된 방식이 아닌, spring.profiles.active=local 환경에서만 spring.config.import=optional:file:.env[.properties] 방식으로 주입하는 Spring Boot 공식 방법을 찾아보세요.
• JPA Batch Insert와 하이버네이트 최적화: 대량의 데이터를 넣을 때 하이버네이트의 hibernate.jdbc.batch_size 설정과 ID 생성 전략(Identity vs Sequence)이 왜 영향을 미치는지 공부해 보시면 큰 도움이 됩니다.

🤔 생각해보기

현재 5만 건의 더미 데이터를 삽입할 때 CommandLineRunner를 사용해 애플리케이션이 켜질 때마다 체크 후 삽입하도록 구성하셨습니다. 만약 실제 실무 서비스 환경(운영 환경)에서도 이 애플리케이션이 켜

💬 리뷰에 대해 궁금한 점이 있나요? 코멘트에 @sparta 를 남겨보세요!
예: @sparta 이 코드에서 동시성 이슈가 발생할 수 있나요?

---

AI 리뷰는 참고용입니다. 최종 판단은 팀원이 직접 합니다.