#106 [RESTRUCTURE] room에 cqrs 패턴 적용 - Githubissues

Wake-up-together-TogetUp / togetup-server

AI를 활용한 커뮤니티 기반 미션 알람 서비스, TogetUp!

1 stars 1 forks source link

#106 [RESTRUCTURE] room에 cqrs 패턴 적용 #119

Closed hye-on closed 7 months ago

hye-on commented 7 months ago

☀️ 작업 사항

room에 cqrs패턴을 적용했습니다.

☀️ 관련 이슈

related #106

☀️ 참고 사항

Criteria로는 조회모델을 커버하는데 한계가 있다고 느껴져 query Dsl을 공부 후 적용할지 고민입니다.
전체적으로 조회모델을 사용하는 도메인에서 사용하지 않는 도메인도 이용하다 보니 어색한 부분도 있는 것 같습니다. 단순한 조회는 JpaRepository를 상속받은 repository에 구현해도 되는데 cqrs패턴에서는 권장하지 않아서 qeuryRepository에 jpql로 구현했습니다.

☀️ 논의 사항

provider를 queryservice로 바꾸고 queryRepository를 만들었습니다. 디스커션에서 여쭈어 보긴 했지만 query가 들어간 이름이 더 직관적이라고 생각해서 바꿨는데 room에는 적절하지만 조회모델이 필요하지 않은 도메인에서는 provider라고 해도 될 것 같다는 생각도 들고, 이름을 통일하는 것이 좋을 것 같다는 생각도 드네요.
찬미님 의견이 궁금합니다.

hye-on commented 7 months ago

승인을 깜빡했네요! 😅

먼저 머지하고 수정사항이 결정되면 pr올리겠습니다! 리뷰 감사합니다 👍