Skip to content
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.

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

account, reaction_history 엔티티 unique index 네이밍 변경 #20

Merged
merged 1 commit into from
Nov 23, 2023

Conversation

oownahcohc
Copy link

@oownahcohc oownahcohc commented Nov 20, 2023

관련 이슈

Detail

account, reaction_history 테이블에 unique index 적용 시, 현재 index_unique_01, index_unique_02 로 네이밍을 했는데,
그래도 좀 더 명확한 이름을 정하는게 좋을 것 같아서 ux_account_social_info, ux_reaction_target 로 변경하려고 합니다.

말씀 주신대로 index 네이밍에 명확한 규칙은 없는 것 같고, 해당 글 에 따르면 접두어-테이블_이름-컬럼_이름 형식으로 보통 표현하는 것 같습니다.

다만 jdbc 에서 ddl 생성 시, 인덱스 명에 kebab case 지원를 허용하지 않는 것 같고, 명확한 규칙이 없는 만큼 각 unique index 가 좀 더 의미를 나타낼 수 있도록 각각 ux_account_social_info, ux_reaction_user_and_target 으로 변경했습니다

@oownahcohc oownahcohc self-assigned this Nov 20, 2023
@oownahcohc oownahcohc merged commit 38a861f into base/20231126 Nov 23, 2023
@oownahcohc oownahcohc deleted the refactor/issue/19 branch November 23, 2023 16:23
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.

2 participants