유저 관련 연관 테이블의 영속성 전이를 수정해줬어요 #249
Merged
Conversation
This file contains 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
taegyun1995
added
chore
taegyun1995
commented
Aug 18, 2023
| @OneToOne(mappedBy = "profileImage", fetch = FetchType.LAZY, cascade = CascadeType.ALL) | ||
| @OneToOne(mappedBy = "pin", fetch = FetchType.LAZY) |
There was a problem hiding this comment.
매핑이 잘못되어있었고, 영속성 전이가 여기엔 필요가 없어서 지웠습니다 ~
| @OneToOne(mappedBy = "profileImage", fetch = FetchType.LAZY, cascade = CascadeType.ALL) | ||
| @OneToOne(mappedBy = "profileImage", fetch = FetchType.LAZY) |
There was a problem hiding this comment.
프로필 이미지에도 마찬가지여서 영속성 전이를 제외시켰습니다 ~
Comment on lines
-53
to
+64
| @OneToOne(fetch = FetchType.LAZY, cascade = CascadeType.ALL) | ||
| @OneToOne(fetch = FetchType.LAZY, cascade = {CascadeType.PERSIST, CascadeType.REMOVE}) | ||
| @JoinColumn(name = "profile_image_id") | ||
| private ProfileImage profileImage; | ||
|
|
||
| @OneToOne(fetch = FetchType.LAZY, cascade = CascadeType.ALL) | ||
| @OneToOne(fetch = FetchType.LAZY, cascade = {CascadeType.PERSIST, CascadeType.REMOVE}) | ||
| @JoinColumn(name = "pin_id") | ||
| private Pin pin; | ||
|
|
||
| @OneToMany(mappedBy = "user", cascade = CascadeType.ALL) | ||
| @OneToOne(mappedBy = "user", fetch = FetchType.LAZY, cascade = CascadeType.REMOVE) | ||
| private RefreshToken refreshToken; |
There was a problem hiding this comment.
프로필 이미지와, 핀은 유저가 생성될 때 같이 생성되게 PERSIST를 적용시켰고
리프레시토큰과 프로필이미지, 핀은 유저가 삭제될 때 다같이 삭제되어야 해서 REMOVE를 적용 시켜서
필요한것만 적용시켰습니다..
Comment on lines
+66
to
+78
| @OneToMany(mappedBy = "user", cascade = CascadeType.REMOVE) | ||
| private List<Link> linkList = new ArrayList<>(); | ||
|
|
||
| @OneToMany(mappedBy = "user", cascade = CascadeType.REMOVE) | ||
| private List<UserHashtag> userHashtagList = new ArrayList<>(); | ||
|
|
||
| @OneToMany(mappedBy = "user", cascade = CascadeType.ALL) | ||
| @OneToMany(mappedBy = "user", cascade = CascadeType.REMOVE) | ||
| private List<Bookmark> bookmarkList = new ArrayList<>(); | ||
|
|
||
| @OneToMany(mappedBy = "user", cascade = CascadeType.ALL) | ||
| @OneToMany(mappedBy = "user", cascade = CascadeType.REMOVE) | ||
| private List<IsLinkRead> isLinkReadList = new ArrayList<>(); | ||
|
|
||
| @OneToMany(mappedBy = "user", cascade = CascadeType.ALL) | ||
| @OneToMany(mappedBy = "user", cascade = CascadeType.REMOVE) |
There was a problem hiding this comment.
연관관계가 삭제될 때만 필요하다 느껴서 바꿔주었습니당..
esubine
approved these changes
Aug 18, 2023
Comment on lines
-53
to
+64
| @OneToOne(fetch = FetchType.LAZY, cascade = CascadeType.ALL) | ||
| @OneToOne(fetch = FetchType.LAZY, cascade = {CascadeType.PERSIST, CascadeType.REMOVE}) | ||
| @JoinColumn(name = "profile_image_id") | ||
| private ProfileImage profileImage; | ||
|
|
||
| @OneToOne(fetch = FetchType.LAZY, cascade = CascadeType.ALL) | ||
| @OneToOne(fetch = FetchType.LAZY, cascade = {CascadeType.PERSIST, CascadeType.REMOVE}) | ||
| @JoinColumn(name = "pin_id") | ||
| private Pin pin; | ||
|
|
||
| @OneToMany(mappedBy = "user", cascade = CascadeType.ALL) | ||
| @OneToOne(mappedBy = "user", fetch = FetchType.LAZY, cascade = CascadeType.REMOVE) | ||
| private RefreshToken refreshToken; |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
개요
유저 관련 연관 테이블의 영속성 전이를 수정해줬어요
📌 관련 이슈
✨ 과제 내용