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

Repo, Pull Request, Comment Aggregate 분리 #18

Open
dks301 opened this issue Jun 11, 2020 · 1 comment
Open

Repo, Pull Request, Comment Aggregate 분리 #18

dks301 opened this issue Jun 11, 2020 · 1 comment
Labels
good first issue Good for newcomers

Comments

@dks301
Copy link
Collaborator

dks301 commented Jun 11, 2020

지금 repository가 다 따로 있는데 하나의 aggregate로 볼 것인지 생각해볼만하지 않을까요?

@dks301 dks301 added the good first issue Good for newcomers label Jun 11, 2020
@ksy90101
Copy link
Collaborator

ksy90101 commented Jun 11, 2020

좋은 의견같아요! 근데, 저번에도 말했듯이 브라운이 했던 이야기와 상충할꺼 같아요.
개념적으로는 한묶음인데, 기술 스택에 의해 어쩔 수 없는 현상이 발생한건데, 이 부분은 제이슨도 트레이드 오프라고 생가한다고 하네요!
그리고 사실 저희는 pullReqeust를 save를 하는데, update를 위해 이렇게 한다고 볼수는 없을꺼 같아요. 그래서 update 쿼리를 만들어서 직접 update를 한다면 진짜로 기술 스택에 의해 어쩔 수 없는 Repository라고 할 수 있지 않을까 생각합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants