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

[refactor] #305 - 릴리즈 1.2.0 버전과 싱크 맞추기 완료 #306

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

hoonyworld
Copy link
Member

Related issue 🛠

Work Description ✏️

  • 릴리즈 1.2.0 버전과 싱크 맞추기를 진행하였습니다. 주요 사항은 다음과 같습니다.

TicketCancelRequest 추가

  • 이 부분은 릴리즈 1.2.0 버전 컨플릭트 해결하면서 생긴 파일인데요, 현재 develop에는 없는 파일입니다.
  • 제 생각에는 TicketCancelRequest가 TicketDeleteRequest로 변한 것 같은데 혹시 맞을까요? (혹시 몰라서 일단 넣는 방향으로 진행했습니다)

ManyToOne -> OneToOne (릴리즈 1.2.0에는 미반영된 상태)

  • OneToOne 관계로 설정된 줄 알았으나, ManyToOne인 걸 발견하여 변경했습니다.
  • 어노테이션 옵션 중 optional=false가 있는데 해당 방식은 외래키를 NotNull 상태로 만들어줍니다.
  • 따라서 저희는 User의 userId를 가져와서 Member 테이블에 같이 저장하는 방식이라서 해당 방식을 도입했습니다.
  • 추가로 optional=false를 설정하게 되면 Inner Join, 사용하지 않으면 left Join이 사용됩니다.
  • 따라서 Inner Join 방식으로 할 때, 필터링 시 필요하지 않은 데이터를 일찍 제거해서 성능에 더 좋을 것 같아 반영했습니다.

참고링크 : https://k1my0unghan.tistory.com/55

Trouble Shooting ⚽️

Related ScreenShot 📷

Uncompleted Tasks 😅

To Reviewers 📢

Copy link
Collaborator

@hyerinhwang-sailin hyerinhwang-sailin left a comment

Choose a reason for hiding this comment

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

TicketCancelRequest가 TicketDeleteRequest로 변한 것 같은데 혹시 맞을까요? -> 정확히 얘기하면 TicketUpdateRequest로 바뀌었습니다!
메이커가 티켓의 예매상태를 예매취소뿐만 아니라 여러 상태로 변경할 수 있게 되면서 update로 네이밍을 바꾸게 되었습니다.
TicketDeleteRequest는 메이커가 티켓 관리 페이지에서 예매자를 삭제할 때의 request dto입니다!

oneToOne으로 member user 관계 바꾸면서 inner join 적용하신 거 좋네요~~
고생하셨습니다!

@hoonyworld hoonyworld merged commit f17d005 into develop Jan 14, 2025
1 check passed
@hoonyworld hoonyworld deleted the refactor/#305 branch January 14, 2025 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] release 1.2.0과 싱크 맞추기
2 participants