Conversation
V3 마이그레이션 대상 데이터의 event.content 최대 길이가 632자로 기존 varchar(500) 한도를 초과해 잘림이 발생한다. session.content(최대 431자)도 여유가 적어 함께 TEXT로 통일한다. Refs: WTH-508 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FUfuUw4qjjHWfZKiyqgADJ
엔티티 컬럼을 TEXT로 넓혔으나 요청 DTO가 500자로 막고 있어, 마이그레이션된 632자 일정을 수정할 때 검증에 걸린다. V3는 해당 필드에 길이 제약이 없었다. Refs: WTH-508 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FUfuUw4qjjHWfZKiyqgADJ
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Flyway로 스키마를 관리하므로 엔티티 변경에 대응하는 V15를 추가한다. prod는 ddl-auto가 validate라 마이그레이션 없이는 기동에 실패한다. Refs: WTH-508 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FUfuUw4qjjHWfZKiyqgADJ
Member
Author
|
핫픽스 PR #108에 흡수되었습니다. 동일 커밋이 두 경로로 들어가면 main→dev 역머지에서 혼란이 생기므로 닫습니다. |
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
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.
배경
V3 → V4 마이그레이션(WTH-508)의 선행 작업입니다.
V3 운영 DB 실측 결과
event.content최대 길이가 632자로, V4의varchar(500)한도를 초과합니다.그대로 마이그레이션하면 본문이 잘립니다.
event.contentmeeting.content→session.contentsession.content는 현재 통과하지만 여유가 69자뿐이고, 컷오버까지 V3가 계속 운영되므로 함께 조정했습니다.Post.content가 이미TEXT라 일관성도 맞습니다.변경 내용
1. 엔티티 컬럼 →
TEXT(d17ce27)2. 요청 DTO 길이 제한 500 → 1000자 (
39fc573)엔티티만 넓히면 마이그레이션된 632자 일정을 수정할 때
@Size(max = 500)에 걸려 저장이 실패합니다.참고로 V3에는 해당 필드에 길이 제약이 없었습니다 (
@Size는 댓글 300자에만 존재, DB는TEXT).무제한 대신 1000자로 제한을 두되 V3 데이터를 충분히 덮도록 했습니다.
ScheduleSaveRequest,ScheduleUpdateRequestSessionCreateRequest,SessionUpdateRequest3. Flyway 마이그레이션
V15(6b780d8)varchar(500)→TEXT는 넓히는 방향이라 기존 데이터는 보존됩니다.검증
./gradlew ktlintCheck compileKotlin통과관련
docs/plan/v3-to-v4-migration-plan.md§2-4, §11🤖 Generated with Claude Code
https://claude.ai/code/session_01FUfuUw4qjjHWfZKiyqgADJ