-
Notifications
You must be signed in to change notification settings - Fork 1
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
refactor: 많이 조회하는 키워드 테이블 제약 조건 추가 #903
refactor: 많이 조회하는 키워드 테이블 제약 조건 추가 #903
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
키워드 로직이 어떻게 구성되어있는지 몰라서 리뷰하기 어려울 것 같아요
오늘 안에 머지되어야 할테니 일단 어프로브 드리겠습니다.
import lombok.Builder; | ||
import lombok.Getter; | ||
import lombok.NoArgsConstructor; | ||
|
||
@Getter | ||
@Entity | ||
@Table(name = "article_search_keyword_ip_map", indexes = { | ||
@Index(name = "idx_ip_address", columnList = "ipAddress") | ||
@Index(name = "idx_ip_address", columnList = "ipAddress")}, uniqueConstraints = { | ||
@UniqueConstraint(name = "ux_keyword_ip", columnNames = {"keyword_id", "ipAddress"}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C
유니크를 걸어둬도 유니크 필드가 중복되면 결국 에러가 발생하는 것 아닌가요? 유니크만 걸어둬도 괜찮나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
동시성 제어 어노테이션으로 나중에 어플리케이션 단에서 막을 것입니다!
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항
원래 동시성 제어 어노테이션이랑 같이 처리 할 생각이였는데 생각보다 동시성 제어 작업이 길어진 점과.. 따로 올리는 것이 낫겠다는 생각이 들어서 급히 올립니다.