Repository Package 위치에 대하여

[hongdosan]

안녕하세요. 모아밤 서버 개발팀의 박세연과 홍혁준입니다.

다름이 아니라, Redis Repository를 사용 중인 Notifiaction과 Token Repository 위치에 대해 논의 드릴려고 디스커션을 작성하게 되었습니다.

---

현재 상황

아래 클래스들은 현재 infrastructure 패키지 내부의 redis 패키지에 위치하고 있습니다.

- StringRedisRepository.java : 해당 클래스는 String Type(opsForValue)의 Redis를 직접적으로 접근하는 레포지토리입니다.
- HashRedisRepository.java : 해당 클래스는 Hash Type(opsForHash)의 Redis를 직접적으로 접근하는 레포지토리입니다.
- NotificationRepository.java : 해당 클래스는 Redis를 직접적으로 사용하지 않고 추상화된 StringRedisRepository.java를 사용하는 알림 서비스 레포지토리입니다.
- TokenRepository.java : 해당 클래스는 Redis를 직접적으로 사용하지 않고 추상화된 HashRedisRepository.java를 사용하는 토큰 서비스 레포지토리입니다.

---

문제 상황

1. NotificationRepository.java와 TokenRepository.java는 사실상 Redis에 직접적으로 접근하지 않음에도 redis package 내부에 위치하여 어색하다 판단했습니다.
2. 다른 Domain Repository와 동일하게 domain package 내부에 위치하는 것은 Token과 Notification이 도메인이 아니기 때문에, 적합하지 않다고 판단했습니다.
3. application package 내부에 위치하는 것은 네이밍상 어색하다고 판단했습니다. 결국에 Repository이기 때문에 네이밍을 변경하기에도 마땅한 네이밍을 찾지 못했습니다.

---

해결 방안

결국에 JPA에 종속된 Domain Repository도 모아밤에서는 MySQL Database과 통신합니다. 그리고 Notification과 Token은 Redis Database와 통신합니다. 결국 이 둘은 Database 종류가 다른 것이지 둘다 외부에 존재하는 Database와 통신하고 있습니다. 그래서 repository 관련 클래스들을 infrastructure 패키지 내부의 repository 패키지에 위치하는 것이 더 적합하지 않을까? 생각해봤습니다.

긴 글 읽어주셔서 감사합니다. 혹시 동의하신다면, 좋아요 부탁드리고 반대 의견이 있으시다면 댓글로 남겨주시기 바랍니다.

[kmebin]

2. 다른 Domain Repository와 동일하게 domain package 내부에 위치하는 것은 Token과 Notification이 도메인이 아니기 때문에, 적합하지 않다고 판단했습니다.

저는 개인적으로 notification도 도메인이라고 생각합니다! token은 auth 도메인에 넣어도 되지 않을까요?