Skip to content

refactor: application 엔티티 미사용 컬럼 제거 #805

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.

Sign up for GitHub

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

Merged
whqtker merged 1 commit into from
Jun 29, 2026
+13 −0
Merged

refactor: application 엔티티 미사용 컬럼 제거 #805

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions src/main/resources/db/migration/V56__drop_legacy_application_choice_columns.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
ALTER TABLE application
DROP FOREIGN KEY fk_application_first_choice_university_info_for_apply_id,
DROP FOREIGN KEY fk_application_second_choice_university_info_for_apply_id,
DROP FOREIGN KEY fk_application_third_choice_university_info_for_apply_id;

DROP INDEX idx_app_first_choice_term_id_search ON application;
DROP INDEX idx_app_second_choice_term_id_search ON application;
DROP INDEX idx_app_third_choice_term_id_search ON application;

ALTER TABLE application
DROP COLUMN first_choice_university_info_for_apply_id,
DROP COLUMN second_choice_university_info_for_apply_id,
DROP COLUMN third_choice_university_info_for_apply_id;
Comment on lines +11 to +13

@chatgpt-codex-connector chatgpt-codex-connector Bot Jun 27, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Backfill choices before dropping legacy columns

In databases that already contain applications created before the application_choice table was introduced, the repo migrations never copy first_choice_university_info_for_apply_id, second_choice_university_info_for_apply_id, or third_choice_university_info_for_apply_id into application_choice (checked V50 through V55, and current queries read only Application.choices). Dropping these columns here permanently discards those users' stored choices; add an INSERT INTO application_choice ... SELECT ... FROM application backfill for each non-null choice order before the drop.

Useful? React with 👍 / 👎.

Loading