Skip to content

Conversation

binimini
Copy link
Contributor

@binimini binimini commented Nov 7, 2022

작업 목록

기본 도메인 코드 작성
  • 엔티티 작성
  • 관련 레포지토리 작성

논의 사항

  • 없음

- Problem - Level : Problem만 단방향으로 연관 가짐
- Problem - ProblemCategory - Category : Problem만 ProblemCategory와 양방향 연관 가짐 (카테고리별 문제 조회 필요할 경우에 Category에 연관 설정할 것)
- Problem 연관 객체 Eager로 가져오도록 설정 (논의 필요)
@binimini binimini added the feature new feature label Nov 7, 2022
@binimini binimini requested a review from JeongA-Shin November 7, 2022 04:41
@binimini binimini self-assigned this Nov 7, 2022
@JeongA-Shin
Copy link
Contributor

개인적인 생각으로는 굳이 eager로 가져와야할 이유가 없다면 lazy로 해 주는 게 더 좋다고 생각합니다.
제가 알기로는 예상하지 못한 SQL이 발생하고, 굉장히 많은 문제를 발생 시킬 가능성 때문에 실무에선 주로 lazy를 디폴트로 사용한다고 알고 있습니다,,, 혹시 의견 어떠신지 궁금합니다!

@JeongA-Shin JeongA-Shin merged commit c065c6c into develop Nov 7, 2022
@binimini binimini deleted the feat/2/basic-entity branch November 8, 2022 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants