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

feat: 위험한 sql문 체크 로직 추가 #1077

Merged
merged 2 commits into from
Nov 27, 2024

Conversation

BaeJinho4028
Copy link
Collaborator

🔥 연관 이슈

🚀 AS-IS

  1. 리팩터링을 하며, 칼럼이나 테이블이 수정, 삭제됨
  2. 기존 WAS에서 이를 반영하지 못하여 새로운 was는 정상적으로 배포는 되지만, 오류 로그가 쏟아져내림.

WAS 배포와 Flyway 배포를 분리하는 것이 맞지만, 일단 수정, 삭제와 같이 문제가되는경우 경우를 다른 변경사항과 격리시킴으로써 이를 해결하고자함. (DROP|TRUNCATE|RENAME|CONSTRAINT|MODIFY|CHANGE) 을 체크

💬 TO-BE

  1. flyway를 제외한 코드가 변경되었는지? -> 이는 수정, 삭제와 같은 flyway만 따로 올릴 수 있음을 목적으로함.
  2. 코드가 변경이 되었다면, 수정,삭제와 같은 위험한 sql을 막음

@BaeJinho4028 BaeJinho4028 added the 기능 새로운 기능을 개발합니다. label Nov 25, 2024
@BaeJinho4028 BaeJinho4028 self-assigned this Nov 25, 2024
Copy link

Unit Test Results

340 tests   339 ✔️  1m 34s ⏱️
  37 suites      1 💤
  37 files        0

Results for commit 1ffd477.

Copy link
Member

@Choi-JJunho Choi-JJunho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋습니다~
해당 내용은 채널에 공유하고, 주간공유같은 때 한번 더 이야기하면 좋을것 같아요

20HyeonsuLee
20HyeonsuLee previously approved these changes Nov 27, 2024
Copy link
Contributor

@20HyeonsuLee 20HyeonsuLee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다~👍
ALTER에 관한 조건이 안보이는것 같습니다. 추가가 필요해보여요

Copy link
Contributor

@20HyeonsuLee 20HyeonsuLee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다~👍
ALTER에 관한 조건이 안보이는것 같습니다. 추가가 필요해보여요

Copy link
Collaborator Author

@BaeJinho4028 BaeJinho4028 left a 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와 같은 옵션을 제한했습니다. 👍

@BaeJinho4028 BaeJinho4028 merged commit 957b4e2 into develop Nov 27, 2024
5 checks passed
@BaeJinho4028 BaeJinho4028 deleted the feature/1076-flyway-modify-delete-validation branch November 27, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 새로운 기능을 개발합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

배포시 Flyway 변경사항을 반영하지 못한 기존 WAS에서 발생하는 오류를 해결한다.
3 participants