Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Member와 Product 프로필 사용 시 OOM 발생 위험성 #198

Merged
merged 4 commits into from
Aug 13, 2024

Conversation

JadeKim042386
Copy link
Member

🔍️ 이 PR을 통해 해결하려는 문제

FastAPI 서버에서 추천 시스템 로직을 수행할 때 OOM이 발생할 수 있는 위험성을 최소화하기 위해 프로필 정보를 DB가 아닌 파일 형식으로 저장하여 활용하도록 수정했습니다. 자세한 내용은 #197 을 참조 바랍니다.

✨ 이 PR에서 핵심적으로 변경된 사항

  • 엔티티 재정의

    • 프로필 정보를 따로 담는 엔티티를 삭제하고 Product, Member, Auction 엔티티 내에 필드를 추가했습니다.
    • 엔티티 생성 시 파일로 저장될 프로필의 파일명은 UUID 형식으로 지정됩니다.
  • 회원가입과 상품 추가/수정 로직 수정

    • 회원가입 시 따로 프로필 저장하는 로직 삭제
    • 상품 추가/수정 시 공통된 메소드를 호출하여 프로필 정보를 담는 파일을 추가 및 덮어쓰
      도록 하고, 이때 fastapi 서버에서의 수정사항에 따라 넘기는 값 중 product_id -> file_name으로 변경
  • 테스트 코드 수정

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분

  • 없음

📌 PR 진행 시 이러한 점들을 참고해 주세요

  • Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
  • Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
  • Review는 특수한 케이스가 아니면 Reviewer로 지정된 시점 기준으로 1일 이내에 진행해 주세요.

Issue Tags

- 프로필 정보를 따로 담는 엔티티를 삭제하고 Product, Member, Auction 엔티티 내에 필드를 추가
- 엔티티 생성 시 파일로 저장될 프로필의 파일명은 UUID 형식으로 지정됨
- 회원가입 시 따로 프로필 저장하는 로직 삭제
- 상품 추가/수정 시 공통된 메소드를 호출하여 프로필 정보를 담는 파일을 추가 및 덮어쓰도록 하고, 이때 fastapi 서버에서의 수정사항에 따라  넘기는 값 중 product_id -> file_name으로 변경
@JadeKim042386 JadeKim042386 added the enhancement New feature or request label Aug 13, 2024
@JadeKim042386 JadeKim042386 self-assigned this Aug 13, 2024
- Profile 관련 테이블을 삭제하고 Product, Auction, Member 테이블 수정
- 테스트 데이터 수정
@JadeKim042386 JadeKim042386 merged commit 53e2d81 into develop Aug 13, 2024
2 checks passed
@JadeKim042386 JadeKim042386 deleted the feature/#197-profile-redefine branch August 13, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Member와 Product 프로필 사용 시 OOM 발생 위험성
1 participant