회원 도메인 개선

[le2sky]

구현 내용

구현 요약

클라이언트에 영향을 끼칠 수 있는 변경 내용

아래 경로의 API에서 사용하는 Dto의 이름을 변경했습니다.

• [POST] /user : 사용자 등록
  • CreateUserResponseDto -> RegisterUserResponse
• [GET] /user/my-page : 사용자 썸네일 조회
  • UserThumbnailResponseDto -> GetUserThumbnailResponse

개선 작업

• 회원 도메인에 헥사고날 아키텍처를 적용
• 단위 테스트, 통합 테스트 개선
• 쿼리 개선

P.S.

커밋의 수와 변경 코드 수가 많아서 죄송합니다. PR 리뷰가 어려울 것 같아서 아래 블로그 글로 설명 대체하겠습니다.

• 레거시 코드를 헥사고날 아키텍처로 전환(1)
• 레거시 코드를 헥사고날 아키텍처로 전환(2)
• 레거시 코드를 헥사고날 아키텍처로 전환(Final)

관련 이슈

118

구현 내용

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

[akalswl14]

모든 코드에 대한 리뷰를 마쳤습니다! 일부 답변을 단 부분이 있으니 확인 부탁드려요!

[le2sky]

@akalswl14 변경분 엄청 많은데.... 리뷰해주셔서 감사합니다! 역시 GOAT .. 피드백해주신 내용 이 PR에 반영해둘게요!!!

그리고 추가 커멘트입니다!

UID에 대한 인덱스 설정 :

varchar 크기를 줄이는 건 생각치 못했네요! naver와 kakao는 제가 따로 벤더-식별자(네이버랑 카카오에서 받아오는데 아마 uid로 추측)로 형식으로 구성해서 사용하고 있는데요. varchar 크기 줄이는 방법 한번 찾아보겠습니다!

도메인 모델과 영속성 분리로 인한 레포지토리 메소드 중복 호출 :

우선 OSIV를 false로 해두어도 저 시점에서는 1차 캐시에 데이터가 남아있을 것으로 생각이 됩니다! 혹시 모르니 오늘 수정하면서 쿼리 찍어볼게요! 원래는 JPA 엔티티에 대해서 더티 체킹을 이용해서 수정하지만, 도메인 자체에 JPA 엔티티가 들어오면 안된다는 가정에는 애플리케이션 레이어에서 로딩 - 도메인 로직 호출 - 상태 저장의 순서로 진행하는 방법 밖에 떠오르지 않았어요!

[akalswl14]

UID에 대한 인덱스 설정 : varchar 크기를 줄이는 건 생각치 못했네요! naver와 kakao는 제가 따로 벤더-식별자(네이버랑 카카오에서 받아오는데 아마 uid로 추측)로 형식으로 구성해서 사용하고 있는데요. varchar 크기 줄이는 방법 한번 찾아보겠습니다!

위 부분은 별도의 이슈를 등록한 뒤에 진행하도록 논의하였습니다!

도메인 모델과 영속성 분리로 인한 레포지토리 메소드 중복 호출 : 우선 OSIV를 false로 해두어도 저 시점에서는 1차 캐시에 데이터가 남아있을 것으로 생각이 됩니다! 혹시 모르니 오늘 수정하면서 쿼리 찍어볼게요! 원래는 JPA 엔티티에 대해서 더티 체킹을 이용해서 수정하지만, 도메인 자체에 JPA 엔티티가 들어오면 안된다는 가정에는 애플리케이션 레이어에서 로딩 - 도메인 로직 호출 - 상태 저장의 순서로 진행하는 방법 밖에 떠오르지 않았어요!

위 부분은 rename 하는 유스케이스의 쿼리를 확인한 결과, select와 update 쿼리가 각 1개씩 수행됨을 확인하여, 별도의 수정 없이 머지하도록 논의하였습니다!