Skip to content

Conversation

@wibaek
Copy link
Member

@wibaek wibaek commented Aug 17, 2024

관련 이슈

작업 내용

학비 지불 방식에 따라 같은 대학이지만 여러 university 가 생성되는 것을 막기 위해, universityInfoForApply 객체에 koreanName 필드를 추가하고 university를 통합시킵니다.

  • university_info_for_apply에 korean_name 필드 추가
  • university의 korean_name을 참조하는 기능들을 university_info_for_apply의 korean_name을 참조하게 바꾸기

특이 사항

테스트와 data.sql 데이터는 수정해두었으나, production 데이터베이스에 올라가있는 데이터는 이후 수동으로 수정 할 예정입니다.

그리고 대학 검색시 키워드 검색에 사용되는 UniversityFilterRepositoryImplcountryOrUniversityContainsKeyword() 에서도 universityInfoForApply.koreanName 가 아닌 university.koreanName 를 사용하고 있지만, 이는 검색 키워드에 사용되는 것이라 university.koreanName로 검색해도 될 것 같습니다. 정확히는 querydsl 사용법을 잘 알지 못해 코드 수정하기가 어렵습니다...

리뷰 요구사항 (선택)

@wibaek wibaek requested review from leesewon00 and nayonsoso and removed request for leesewon00 and nayonsoso August 17, 2024 06:14
@leesewon00 leesewon00 requested review from leesewon00 and nayonsoso and removed request for leesewon00 and nayonsoso August 17, 2024 07:01
@wibaek wibaek merged commit 7cf69cd into solid-connection:main Aug 23, 2024
@wibaek
Copy link
Member Author

wibaek commented Aug 23, 2024

// 임시 Nullable 필드 생성
ALTER TABLE university_info_for_apply ADD COLUMN korean_name VARCHAR(100);

// university에서 koreanName 가져오기
UPDATE university_info_for_apply
SET korean_name = (
SELECT u.korean_name
FROM university u
WHERE u.id = university_info_for_apply.university_id
);

// Not null 설정
ALTER TABLE university_info_for_apply
MODIFY COLUMN korean_name VARCHAR(255) NOT NULL;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

university와 university_info_for_apply 테이블 정규화

2 participants