Skip to content

Conversation

joel-you
Copy link
Collaborator

@joel-you joel-you commented Sep 5, 2025

… eviction and event publishing

📋 PR 요약

  • 상품 삭제 api 제공
  • 필드 추가
  • 기존 테스트코드 수정

🔗 관련 이슈

#25

  • Closes #이슈번호
  • Related to #이슈번호

📝 변경 사항

  • 새로운 기능 추가
  • 버그 수정
  • 리팩토링
  • 성능 개선
  • 문서 업데이트
  • 테스트 코드 추가/수정
  • 기타:

🛠️ 기술적 변경 사항

  • 수정된 파일:
  • 새로 추가된 파일:
  • 제거된 파일:
  • 주요 로직 변경:

🧪 테스트

  • 단위 테스트 통과
  • 통합 테스트 통과
  • E2E 테스트 통과 (해당하는 경우)
  • 수동 테스트 완료

테스트 방법:

📊 성능 영향

  • 성능 향상 있음
  • 성능에 영향 없음
  • 성능 저하 가능성 있음 (사유: )

🔒 보안 체크리스트

  • 새로운 의존성 추가 시 보안 검토 완료
  • 사용자 입력 검증 로직 확인
  • 인증/권한 로직 검토 완료
  • 민감한 정보 노출 위험 없음

🔄 데이터베이스 변경

  • 데이터베이스 스키마 변경 없음
  • Migration 파일 추가됨
  • 기존 데이터와 호환됨
  • 롤백 계획 수립됨

📱 호환성

  • 이전 버전과 호환됨
  • Breaking Change 없음
  • API 변경 없음
  • 환경 설정 변경 필요 (상세: )

📖 문서 업데이트

  • README.md 업데이트 완료
  • API 문서 업데이트 완료
  • 코드 주석 업데이트 완료
  • 별도 문서 업데이트 불필요

🎯 배포 고려사항

  • 운영 환경 배포 가능
  • 설정 파일 변경 필요
  • 서버 재시작 필요
  • 데이터 마이그레이션 필요
  • 특별한 배포 순서 필요 (상세: )

📸 스크린샷 (UI 변경 시)

Before:

After:

🔍 리뷰 포인트

✅ 체크리스트

  • 코드가 컨벤션을 따르고 있습니다
  • 자가 리뷰를 완료했습니다
  • 적절한 라벨을 추가했습니다
  • 관련 문서를 업데이트했습니다
  • 테스트를 추가/업데이트했습니다
  • CI/CD 파이프라인이 통과합니다

💬 추가 메모

@joel-you joel-you requested a review from flab-true September 5, 2025 18:31
@joel-you joel-you self-assigned this Sep 5, 2025
@joel-you joel-you added ✨ Feature 새로운 API 또는 기능 개발 🌐 API REST API, GraphQL 관련 작업 ⚠️ High Priority 높은 우선순위 labels Sep 5, 2025
@flab-true
Copy link
Collaborator

https://github.com/f-lab-edu/msa-commerce-lab/blob/feature/25-remove-products/common/src/main/java/com/msa/commerce/common/config/RedisConfig.java#L78
GenericJackson2JsonRedisSerializer 사용한 이유가 무엇인가요? Java에서 기본적으로 제공하는 직렬화 방식이 아닌 왜 json 직렬화를 권장할까요?

@flab-true
Copy link
Collaborator

flab-true commented Sep 10, 2025

https://github.com/f-lab-edu/msa-commerce-lab/blob/feature/25-remove-products/buildSrc/src/main/java/Dependency.java#L7
여기에서 Redis Client는 무엇과 의존성을 가지고 있나요? Jedis vs Lettuce? 성능적인 부분에 대한 차이점을 조사해 주세요.

Copy link
Collaborator

@flab-true flab-true left a comment

Choose a reason for hiding this comment

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

간단한 코멘트 남깁니다. :-)

Copy link

Copy link
Collaborator

@flab-true flab-true left a comment

Choose a reason for hiding this comment

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

LGTM 👍 수고 많으셨습니다.

@joel-you joel-you merged commit f509cf6 into develop Sep 21, 2025
2 checks passed
joel-you added a commit that referenced this pull request Sep 22, 2025
feat(25): Implement soft delete functionality for products with cache…
joel-you added a commit that referenced this pull request Sep 23, 2025
feat(25): Implement soft delete functionality for products with cache…
@joel-you joel-you deleted the feature/25-remove-products branch September 23, 2025 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌐 API REST API, GraphQL 관련 작업 ✨ Feature 새로운 API 또는 기능 개발 ⚠️ High Priority 높은 우선순위
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants