🩹 [STMT-62] 스터디 도메인 누락된 컬럼들 추가

[05AM]

💁 해결 하려는 문제를 적어주세요

• 이전 스터디 도메인에 누락된 컬럼이 있었습니다.

🤔 어떤 방식으로 해결했는지 적어주세요

• 누락된 컬럼인 정기모임 시간과 정기모임 반복을 추가했습니다.
• 정기모임 반복은 반복 주기인 매일/매주/매달과 선택한 요일/날짜를 구분자 ;로 구분하는 문자열로 저장하려고 합니다.
  ex) WEEKLY;월요일;수요일;금요일;

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

• 변환 로직을 Mapper와 Domain 내부 구현에 관해서

  • 구현 과정에서 DB에서 불러온 구분자로 연결된 문자열을 변환하는 과정을 mapper에게 위임할지 도메인에서 처리할지 고민했습니다.
  • 결론적으로는 첫번째로 Repeat 클래스를 StudyMeetingSchedule 내부에서만 사용하는 private 클래스로 만들었기 때문에 mapper에서 생성할 수 없었고, 두번째로 첫번째 요소는 반복 유형, 두번째 이후로는 날짜/요일을 저장하는 방식 또한 제가 결정한 도메인 규칙이라고 생각하여 mapper보다는 도메인 내부에서 구현하였습니다.
  • StudyMeetingSchedule의 생성자 내부의 조금 복잡한 로직을 메서드 추출할까 고민도 했지만, 재사용할 일이 없다고 생각하여 굳이 하지 않았습니다.
  • 위의 결정에 대해 어떻게 생각하시는지 궁금합니다! 🤔

• flyway sql문 수정

  • 누락된 부분을 이전 study 테이블 ddl 파일에 직접 수정했는데, 그 이유는 어차피 study의 dummy data도 달라지므로 스키마 히스토리를 지워야하기 때문에 깔끔하게 수정했습니다. 혹시 이에 대한 의견이 있으신지 궁금합니다.
  • 또한 ddl의 경우 이를 수정하는 새로운 sql 파일을 만들어 수정이 가능하지만, dml은 그럴 수 없는 것 같습니다.
  • 따라서 dml은 flyway의 버저닝 파일로 추가하지 않고 data.sql 등의 파일로 따로 만들어서 git으로 형상을 관리하는 방법을 도입하고 싶은데 이에 대한 의견도 궁금합니다!

• 참고) db 작업을 마쳐야 ci action이 성공할 것 같습니다.

🧑‍🏫 이해를 위해 필요한 자료가 있다면 첨부해주세요

[zxcv9203]

• 변환 로직을 Mapper와 Domain 내부 구현에 관해서

  • 구현 과정에서 DB에서 불러온 구분자로 연결된 문자열을 변환하는 과정을 mapper에게 위임할지 도메인에서 처리할지 고민했습니다.
  • 결론적으로는 첫번째로 Repeat 클래스를 StudyMeetingSchedule 내부에서만 사용하는 private 클래스로 만들었기 때문에 mapper에서 생성할 수 없었고, 두번째로 첫번째 요소는 반복 유형, 두번째 이후로는 날짜/요일을 저장하는 방식 또한 제가 결정한 도메인 규칙이라고 생각하여 mapper보다는 도메인 내부에서 구현하였습니다.
  • StudyMeetingSchedule의 생성자 내부의 조금 복잡한 로직을 메서드 추출할까 고민도 했지만, 재사용할 일이 없다고 생각하여 굳이 하지 않았습니다.
  • 위의 결정에 대해 어떻게 생각하시는지 궁금합니다! 🤔

저는 개인적으로 구분자를 통해 데이터를 영속화하고 가져오는 것 자체가 도메인 규칙이라기보다는 DB에서 쉽게 영속화하기 위해 데이터 포맷을 정한거에 가깝다고 생각이 들어 mapper 쪽에 둘 것 같습니다.
저는 도메인에 요일에 대한 List만 존재하고 이를 가져오는게 좀 더 자연스럽다고 생각합니다.

데이터를 어떻게 저장하는지는 상황에 따라 계속 바뀔 수 있는데 도메인에서는 요일에 대한 정보가 필요하지 이를 어떻게 파싱하는지에 대한 정보는 불필요하다고 생각이 듭니다.
구분자를 통한 문자열이 비즈니스 로직과 무관하기 때문에 찬미님께서 Repeat라는 클래스를 만들고 이를 파싱하는 생성자 로직을 추가했다는 생각이 듭니다.
만약 구분자를 포함한 문자열이 비즈니스 로직에 영향을 끼친다면 그 문자열 자체로 존재해야 하지 않나라는 생각이 듭니다.

• flyway sql문 수정

  • 누락된 부분을 이전 study 테이블 ddl 파일에 직접 수정했는데, 그 이유는 어차피 study의 dummy data도 달라지므로 스키마 히스토리를 지워야하기 때문에 깔끔하게 수정했습니다. 혹시 이에 대한 의견이 있으신지 궁금합니다.
  • 또한 ddl의 경우 이를 수정하는 새로운 sql 파일을 만들어 수정이 가능하지만, dml은 그럴 수 없는 것 같습니다.
  • 따라서 dml은 flyway의 버저닝 파일로 추가하지 않고 data.sql 등의 파일로 따로 만들어서 git으로 형상을 관리하는 방법을 도입하고 싶은데 이에 대한 의견도 궁금합니다!

• 참고) db 작업을 마쳐야 ci action이 성공할 것 같습니다.

개인적으로는 study에 dummy data를 넣는 사례자체가 지양되어야할 것 같다고 생각합니다.
이러한 더미 데이터는 테스트에서 관리해야하는 데이터이지 실제 운영에서 포함되기에는 부적절하다고 생각이듭니다 🤔

data.sql을 사용해도 문제 없을 것 같은데 실제 제대로 동작하는지는 테스트해봐야할 것 같습니다.
flyway와 data.sql 중 누가 우선순위를 가지는지 저는 잘 몰라서..!

[05AM]

저는 개인적으로 구분자를 통해 데이터를 영속화하고 가져오는 것 자체가 도메인 규칙이라기보다는 DB에서 쉽게 영속화하기 위해 데이터 포맷을 정한거에 가깝다고 생각이 들어 mapper 쪽에 둘 것 같습니다. 저는 도메인에 요일에 대한 List만 존재하고 이를 가져오는게 좀 더 자연스럽다고 생각합니다.

그렇군요 저도 동의하는 부분입니다. 사실 내부 클래스의 생성을 바깥에서 하는게 맞는가에 대한 고민에 마음이 도메인에서 구현하는 것으로 기울었던 것 같습니다. 해당 부분 반영하여 커밋 올리겠습니다.

추가로 말씀드리자면 매주의 경우 리스트에 요일 문자열을 저장하지만, 매달의 경우 날짜를 저장하게 됩니다.
따라서 반복 유형에 대한 정보도 필요한 것 같습니다.

개인적으로는 study에 dummy data를 넣는 사례자체가 지양되어야할 것 같다고 생각합니다. 이러한 더미 데이터는 테스트에서 관리해야하는 데이터이지 실제 운영에서 포함되기에는 부적절하다고 생각이듭니다 🤔

data.sql을 사용해도 문제 없을 것 같은데 실제 제대로 동작하는지는 테스트해봐야할 것 같습니다. flyway와 data.sql 중 누가 우선순위를 가지는지 저는 잘 몰라서..!

제가 생각하는 data.sql은 flyway에 의해 관리되는 파일은 아니고, 스터디 분야나 멤버의 직업 등의 insert 문을 모아두고 수동으로 넣어주는 방법을 생각했습니다.
pr에서 언급한 것처럼 테이블 구조와 다르게 데이터는 구조 변경에 대응하기 어려운 것 같아서 history schema에 저장되지 않아 상대적으로 수정이 편한 방식으로 관리하면 좋을 것 같다고 생각했습니다!

[zxcv9203]

제가 생각하는 data.sql은 flyway에 의해 관리되는 파일은 아니고, 스터디 분야나 멤버의 직업 등의 insert 문을 모아두고 수동으로 넣어주는 방법을 생각했습니다. pr에서 언급한 것처럼 테이블 구조와 다르게 데이터는 구조 변경에 대응하기 어려운 것 같아서 history schema에 저장되지 않아 상대적으로 수정이 편한 방식으로 관리하면 좋을 것 같다고 생각했습니다!

아 Spring에서 지원하는 sql init 기능을 활용하는 data.sql을 말씀하신게 아니었나보군요

수동으로 넣어줘도 되지만 저는 좀 번거로운거 같아서 ㅎㅎ.. 자동으로 했을때 불편하다고 느끼는거면 버전 관리 정도만 해주셔도 좋을 것 같네요 👍

[05AM]

제가 생각하는 data.sql은 flyway에 의해 관리되는 파일은 아니고, 스터디 분야나 멤버의 직업 등의 insert 문을 모아두고 수동으로 넣어주는 방법을 생각했습니다. pr에서 언급한 것처럼 테이블 구조와 다르게 데이터는 구조 변경에 대응하기 어려운 것 같아서 history schema에 저장되지 않아 상대적으로 수정이 편한 방식으로 관리하면 좋을 것 같다고 생각했습니다!

아 Spring에서 지원하는 sql init 기능을 활용하는 data.sql을 말씀하신게 아니었나보군요

수동으로 넣어줘도 되지만 저는 좀 번거로운거 같아서 ㅎㅎ.. 자동으로 했을때 불편하다고 느끼는거면 버전 관리 정도만 해주셔도 좋을 것 같네요 👍

아하 처음 안 내용인데 관련해서 테스트해보고 자동으로 적용하는 코드를 구현한 PR을 올려보겠습니다!