fix(#311): entity 이름 변경: user_tag -> tag_user, brand_tag -> tag_brand#312
fix(#311): entity 이름 변경: user_tag -> tag_user, brand_tag -> tag_brand#312Yoonchulchung merged 3 commits intodevfrom
Conversation
Summary of ChangesHello @Yoonchulchung, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
이 PR은 user_tag를 tag_user로, brand_tag를 tag_brand로 엔티티 이름을 변경하는 작업을 성공적으로 수행했습니다. Java 엔티티, 리포지토리, 서비스뿐만 아니라 Python 데이터 생성 스크립트에 이르기까지 코드베이스 전반에 걸쳐 일관성 있게 수정되었습니다. 이를 통해 코드의 명확성과 일관성이 향상되었습니다.
한 가지 작은 개선점을 발견했는데, 주석 처리된 Python 스크립트에서 이전 테이블 이름이 여전히 사용되고 있는 부분입니다. 이를 수정하면 나중에 해당 코드를 다시 활성화할 때 발생할 수 있는 잠재적인 문제를 예방할 수 있습니다.
| # # user_tag 매핑 정보 조회 | ||
| # cursor.execute(""" | ||
| # SELECT ut.user_id, ut.tag_id, t.tag_type, t.tag_category | ||
| # FROM user_tag ut |
Summary
Tag_User 엔티티에 isDeprecated가 추가되면서 DB에 반영되지 않아 조회에서 문제가 발생.
DB를 업데이트하고, 도메인 별로 다른 Entity라는 것을 명시하기 위해 이름 변경.
Changes
엔티티 수정:
BrandService 수정:
Type of Change
Related Issues
#311