setting: apply flyway db tool #14
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chominho96
reviewed
Jan 29, 2024
| dividend integer not null, | ||
| ex_dividend_date datetime(6) not null, | ||
| payment_date datetime(6) not null, | ||
| stock_id binary(16) not null, |
There was a problem hiding this comment.
지금처럼 간접 참조를 할 때, 외래키 제약 조건을 걸지 않아도 될까요?!
이 부분은 궁금해서 좀 찾아봤는데, 의견이 나뉘더라구요?!
- 외래키 제약 조건을 걸지 않으면 성능 튜닝에 유리하고, 복잡도를 줄인다
- 외래키 제약 조건을 걸면 휴먼 에러를 줄일 수 있어 DB 관리에 용이하다 + 성능도 극한 튜닝 아니면 큰 상관이 없다!
이정도인거 같아요!!
There was a problem hiding this comment.
규모가 큰 시스템이라면 사실 모든 테이블에 제약 조건을 걸기에는 무리가 있더라구요..!! test fixture를 만들 때도 꽤나 번거롭고
cascade 가 걸려있다면 예상치 못하게 데이터 변경이 일어나기도 하고요!!
규모가 큰 서비스였다면 반드시 생명주기가 같아야 하는 테이블이 아니고서야 걸지 않는 것을 선호합니당
하지만 저희는 테이블이 2개밖에 없으니,,,, 데이터 안정성 측면에서 걸어도 좋다고 생각합니다ㅎㅎㅎㅎ 추가해놓을게요 👍
There was a problem hiding this comment.
오.. 그렇군요!! test fixture 만들 때 제약 조건이 있으면 번거로웠던 경험은 있었는데 데이터 변경으로 인해 큰 서비스에서는 선호하지 않는다는 것은 처음 알았네요!! 감사합니다 👍
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue Number
close: #13
작업 내역
DB 테이블 버전 관리를 위해 flyway 적용
변경사항
PR 특이 사항