-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: 위험한 sql문 체크 로직 추가 #1077
feat: 위험한 sql문 체크 로직 추가 #1077
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋습니다~
해당 내용은 채널에 공유하고, 주간공유같은 때 한번 더 이야기하면 좋을것 같아요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~👍
ALTER에 관한 조건이 안보이는것 같습니다. 추가가 필요해보여요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~👍
ALTER에 관한 조건이 안보이는것 같습니다. 추가가 필요해보여요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@20HyeonsuLee ALTER를 막으면 테이블을 새로 생성하는것이 아니라면 칼럼 추가를 할수가 없어서, ALTER 대신 DROP, MODIFY와 같은 옵션을 제한했습니다. 👍
🔥 연관 이슈
🚀 AS-IS
WAS 배포와 Flyway 배포를 분리하는 것이 맞지만, 일단 수정, 삭제와 같이 문제가되는경우 경우를 다른 변경사항과 격리시킴으로써 이를 해결하고자함. (DROP|TRUNCATE|RENAME|CONSTRAINT|MODIFY|CHANGE) 을 체크
💬 TO-BE