Skip to content

Conversation

@move-hoon
Copy link
Member

@move-hoon move-hoon commented Nov 15, 2025

🔗 관련 이슈

📘 작업 유형

  • ✨ Feature (기능 추가)
  • 🐞 Bugfix (버그 수정)
  • 🔧 Refactor (코드 리팩토링)
  • ⚙️ Chore (환경 설정)
  • 📝 Docs (문서 작성 및 수정)
  • ✅ Test (기능 테스트)
  • 🎨 style (코드 스타일 수정)

📙 작업 내역

문제 원인:

  • MySQL 8.0은 ALTER TABLE ADD COLUMN 구문에서 IF NOT EXISTS를 지원하지 않습니다. 이 구문은 PostgreSQL과 MariaDB 10.0+ 에서만 지원됩니다.

수정 내용:

  • 두 개의 ALTER TABLE 구문에서 IF NOT EXISTS 절을 제거했습니다.

flyway_schema_history 에서 해당 마이그레이션 튜플 제거

  • Flyway가 V20251116_001을 아직 실행되지 않은 새로운 마이그레이션으로 인식하도록 튜플을 제거하였습니다

🧪 테스트 내역

  • 브라우저/기기에서 동작 확인
  • 엣지 케이스 테스트 완료
  • 기존 기능 영향 없음

🎨 스크린샷 또는 시연 영상 (선택)

기능 미리보기 기능 미리보기
기능 설명 기능 설명

✅ PR 체크리스트

  • 커밋 메시지가 명확합니다
  • PR 제목이 컨벤션에 맞습니다
  • 관련 이슈 번호를 작성했습니다
  • 기능이 정상적으로 작동합니다
  • 불필요한 코드를 제거했습니다

💬 추가 설명 or 리뷰 포인트 (선택)

  • ..

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 데이터베이스 마이그레이션 구성을 조정했습니다.

@move-hoon move-hoon self-assigned this Nov 15, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 15, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

MySQL 8.0 호환성을 위해 Flyway 마이그레이션 SQL 파일에서 ALTER TABLE ADD COLUMN 구문의 IF NOT EXISTS 절을 제거했습니다. device 및 notification 테이블의 deleted_at 칼럼 추가 구문이 수정되었습니다.

Changes

응집 / 파일 변경 요약
MySQL Flyway 마이그레이션 수정
infra/src/main/resources/db/migration/mysql/V20251116_001__add_deleted_at_to_device_notification.sql
device 및 notification 테이블의 ADD COLUMN 구문에서 IF NOT EXISTS 절 제거. MySQL 8.0 호환성을 위한 구문 수정

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 분

  • 단순한 SQL 구문 수정
  • 조건부 문절 제거만 포함
  • 제어 흐름이나 로직 변경 없음
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch BOOK-441-fix/#138

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 15755d4 and 00381c9.

📒 Files selected for processing (1)
  • infra/src/main/resources/db/migration/mysql/V20251116_001__add_deleted_at_to_device_notification.sql (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot requested a review from minwoo1999 November 15, 2025 16:51
@move-hoon move-hoon merged commit 1167e92 into develop Nov 15, 2025
3 of 4 checks passed
@sonarqubecloud
Copy link

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.

[BOOK-441/fix] flyway SQL 구문 오류 수정

2 participants