Conversation
Summary of ChangesHello @w0uldy0u, 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은 마인드맵 내 역량 태그 조회 시 발생하던 서버 오류를 해결하는 데 중점을 둡니다. 기존 JPQL 쿼리의 한계를 극복하기 위해 네이티브 SQL 쿼리를 도입하여 데이터 조회 로직을 개선했으며, 관련 테스트 코드를 리팩토링하여 견고성을 확보했습니다. Highlights
Changelog
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은 마인드맵 내 역량 태그 조회 API에서 발생하던 500 에러를 수정하고, 관련 테스트 코드를 리팩토링하는 작업을 포함하고 있습니다. JPQL에서 지원하지 않는 JSON 타입 필드 조회를 위해 네이티브 쿼리와 JSON_TABLE 함수를 사용한 해결 방법은 적절합니다. 또한, 테스트 코드 전반에 걸쳐 TestEntityFactory를 도입하고 중복 코드를 제거하여 테스트의 가독성과 유지보수성을 크게 향상시킨 점이 인상적입니다. AbstractRepositoryTest에서 Testcontainers 설정을 개선한 부분도 좋은 변경입니다. 전반적으로 코드 품질을 높이는 좋은 변경이라고 생각합니다. 한 가지 작은 개선점을 TestEntityFactory에 제안했습니다.
kimhji
left a comment
There was a problem hiding this comment.
LGTM 💯
오류 나는 부분 처리해주셔서 감사합니다 😄
Closes #367
목적
500에러를 일으켰던 기존 마인드맵 내부 역량 태그 조회 API를 고칩니다.
작업 내용
결과