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: Flyway DDL 파일 추가 (#11) #24

Merged
merged 3 commits into from
Dec 14, 2023
Merged

feat: Flyway DDL 파일 추가 (#11) #24

merged 3 commits into from
Dec 14, 2023

Conversation

seokjin8678
Copy link
Contributor

관련 이슈

PR 세부 내용

지금까지 정의된 엔티티에 대해 DDL 파일을 추가했습니다.

Language, NewsType과 같은 Enum 타입은 @Enumerated(EnumType.STRING) 어노테이션을 사용하여 DB에 varchar 타입으로 저장되게 하였지만, 하이버네이트 schema validation에서 enum 타입이 아니라고 예외를 던지더군요. 😂
이전 프로젝트에선 이런 일이 없어 당황스러웠네요.
우선 columnDefinition = "varchar" 속성으로 해결했습니다.

또한 Content의 content 컬럼은 원래 BLOB(LONGTEXT) 이었지만, 뉴스의 길이가 64KB를 넘기지 않을 것 같아 TEXT로 타입을 설정했습니다.
나중에 번역된 뉴스의 길이를 보고 BLOB으로 변경할 수 있어야 할 것 같네요.

또한 local 프로파일에 MySQL, Flyway, 로깅 등 설정을 추가했습니다.
local 프로파일은 개인 마다 설정을 다르게 할 필요가 있으므로 .gitignore에 등록하여 깃의 staging 대상에 들어가지 않도록 하는 방법도 좋아보이네요.

- enum 타입 varchar 타입 명시
- content 컬럼 lob -> text 변경
- originUrl -> origin_url
@seokjin8678 seokjin8678 added 뉴스 뉴스 도메인 🏢 인프라 인프라에 관한 작업 ☢️ DB 데이터베이스에 관한 작업 labels Dec 13, 2023
@seokjin8678 seokjin8678 requested a review from Laeng December 13, 2023 16:44
@seokjin8678 seokjin8678 self-assigned this Dec 13, 2023
@seokjin8678 seokjin8678 linked an issue Dec 13, 2023 that may be closed by this pull request
Copy link

Test Results

  5 files    5 suites   0s ⏱️
17 tests 17 ✔️ 0 💤 0
18 runs  18 ✔️ 0 💤 0

Results for commit de08bc4.

Copy link
Member

@Laeng Laeng left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다!

@seokjin8678 seokjin8678 merged commit 240bcb1 into dev Dec 14, 2023
3 checks passed
@seokjin8678 seokjin8678 deleted the feat/#11 branch December 15, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
뉴스 뉴스 도메인 ☢️ DB 데이터베이스에 관한 작업 🏢 인프라 인프라에 관한 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Flyway DDL 파일을 추가한다.
2 participants