refactor: 사용하지 않을 컬럼 제거 #186
Merged
Conversation
This file contains hidden or 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
Closed
2 tasks
nayonsoso
approved these changes
Feb 7, 2025
Collaborator
There was a problem hiding this comment.
오.. 저는 규혁님 PR 받아서 로컬에서 실행했는데 ddl-auto:update 여도 실행됐습니다.
혹시 profile 을 dev 로 설정하고, sql.init.mode: never 도 설정하셨나요?
아니라면 어떤 메세지 때문에 실행 안되는지 공유해주세요!
그리고 GpaScoerRequest, GpaScoreStatus, LanguageTestScoreRequest, LanguageTestScoreStatus 에
안쓰는 LocalDate 임포트 없애주세요!
Contributor
Author
|
아 정확히는 never로 설정하면 issueDate 드랍하는 쿼리문이 안뜨고 설정안하면 data.sql때문에 에러터지는 거였는데 never로 설정하면 issueDate 드랍하는 쿼리문이 왜 안나오는지 모르겠더라구요
반영했습니다! |
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.
관련 이슈
작업 내용
GpaScore 과 LanguageTestScore 의 사용하지 않을 컬럼인 "issueDate"를 없앴습니다.
특이 사항
Flyway 이용하려고 Hibernate SQL 사용할 때 ddl-auto update로 사용하시나요? 저는 그렇게하면 서버가 안켜지고 create로 하니까 그냥 alter로 issueDate를 드랍하는 게 아니라 그냥 새 테이블 생성 쿼리문 주더라구요..
이번엔 컬럼 드랍하는 간단한 쿼리문이여서 그냥 제가 작성했습니다 @wibaek