Skip to content

fix: 대학 검색 키워드 시드 마이그레이션 보정 - #659

Merged
JanooGwan merged 1 commit into
developfrom
fix/university-search-keyword-v87
Jun 1, 2026
Merged

JanooGwan merged 1 commit into
developfrom
fix/university-search-keyword-v87

Conversation

@JanooGwan

Copy link
Copy Markdown
Contributor

개요

  • stage 배포에서 V87__seed_university_search_keywords.sql이 일부 대학만 존재하는 DB에서 실패한 문제를 보정합니다.
  • Flyway가 실패한 V87을 먼저 검증하기 때문에, V88 후속 마이그레이션이 아니라 실패한 V87 자체가 재실행 가능해야 합니다.

주요 변경 내용

  • V87 seed 쿼리를 LEFT JOIN에서 JOIN으로 변경해 실제 존재하는 대학 키워드만 insert합니다.
  • ON DUPLICATE KEY UPDATE를 추가해 repair 이후 재시도나 중복 데이터가 있어도 안전하게 처리합니다.
  • V88 후속 마이그레이션은 제거했습니다.
  • 한국기술교육대학교만 있는 DB에서 V87이 3개 키워드만 seed하는 테스트로 변경했습니다.

배포 참고

  • stage DB에는 V87 실패 이력이 남아 있으므로, 새 코드 배포 전/중에 flyway repair로 실패 이력을 제거해야 합니다.
  • repair 후 새 V87 파일이 실행되면 한국기술교육대학교 키워드만 seed되고 애플리케이션이 다음 단계로 진행됩니다.

검증

  • ./gradlew.bat test --tests "gg.agit.konect.unit.domain.university.UniversitySearchKeywordMigrationTest"
  • ./gradlew.bat checkstyleMain

Checklist

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함
  • 보안 및 민감 정보 검증(API 키, 환경 변수, 개인정보 등)

@coderabbitai

coderabbitai Bot commented Jun 1, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@JanooGwan, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 3 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: f6c3a588-b5cd-4848-bf04-7bcd69cdae58

📥 Commits

Reviewing files that changed from the base of the PR and between 4a70e4e and 8fb5f4b.

📒 Files selected for processing (3)
  • src/main/resources/db/migration/V87__seed_university_search_keywords.sql
  • src/main/resources/db/migration/V88__seed_existing_university_search_keywords.sql
  • src/test/java/gg/agit/konect/unit/domain/university/UniversitySearchKeywordMigrationTest.java
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/university-search-keyword-v87

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.

@JanooGwan JanooGwan added the 버그 정상적으로 동작하지 않는 문제 상황 관련 이슈입니다. label Jun 1, 2026
@JanooGwan JanooGwan self-assigned this Jun 1, 2026
@JanooGwan
JanooGwan merged commit 6be8eb1 into develop Jun 1, 2026
5 checks passed
@JanooGwan
JanooGwan deleted the fix/university-search-keyword-v87 branch June 1, 2026 03:17
@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown

🧪 JaCoCo Coverage Report (Changed Files)

이 PR에서 변경된 main Java 소스 파일이 없습니다.

📊 View Workflow Run

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.

1 participant