Open
Conversation
[feat] 나의 팀 리스트 조회 구현
[fix] 로컬/배포 트랜잭션 LazyInitializationException 해결
[hotfix] authDetails null 에러 처리
[hotfix] redis 에러 처리
[hotfix] redis set 에러 처리
Added images and detailed explanations of features, performance improvements, and project structure.
Added detailed descriptions for various technologies used in the project, including Java, Spring Boot, Docker, and more.
[feat] uuid로 프로필 id 리스트 조회 API
[fix] 다른 유저 프로필 리스트 조회 API
[feat] 팀피셜록 신고하기 API
pr Feat/지원자 현황 조회 응답에 대표 키워드 리스트 추가
[feat] 신고 시 이메일 전송 및 어드민 신고관리 API 구현
[fix] 신고 중복 문제 해결
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[오피셜팀] 팀피셜 백엔드 코드리뷰
✨ 리뷰를 요청드리는 주요 부분
가장 고민되는 지점을 아래 형식으로 정리했습니다.
Q1. 제가 현재 서비스 책임을 생각하면서 구현중인데, 이 방식이 좋은 방법인가 싶어 여쭤봅니다.
예를 들어 mypage에서 application을 찾는 로직을 구현할 때 ApplicationRepository를 이용해야하는데 다른 도메인 서비스에서도 많이 쓰이기 때문에 재활용 측면에서 ApplicationService에 해당 메소드를 만들고 사용하는 방식으로 하고있습니다. 그리고 같은 도메인의 서비스에서만 repo에 접근할 수 있게 구현중이에요
여기서 든 생각은 필터링 로직을 추가한 지원 조회 등 한 api에서만 쓰는 repository 로직도 같은 도메인에서 구현하는 게 맞나싶습니다.. (재활용될 일도 없는데 서비스를 쓸데없이 2번 타고 들어간다는 단점)
그래서 해당 부분을 그냥 MypageService에서 ApplicationRepository를 바로 쓰는 게 나을지 싶습니다!
domain/application/service/MypageService.javadomain/application/service/ApplicationService.javadomain/application/repository/ApplicationRepository.javaQ2. QueryDSl를 이용해서 필터링 로직을 구현했는데, 페이징 처리를 같이하다보니 Repository자체에서 dto projection을 통해서 성능문제를 없애려고 했는데, List필드가 있다보니 코드가 길어지는 문제가 있는것 같습니다. 혹시 더 좋은 방법이 있는지 궁금합니다 !
domain/application/repository/RecruitingPostRepositoryImpl.java💬 오피셜팀이 고민하고 있는 부분
Q1. 게시글 모집 기간 마감을 구현하기 위해 현재 Spring batch를 사용할 예정입니다. 기존에는 API서버안에 함께 구현을 했었는데 그렇게 하면 API서버 성능 문제가 발생한다고 해서, 이번에는 모집 공고의 ‘모집 기간 마감’을 자동으로 처리하기 위해 Spring Batch 기반 배치 서버를 별도로 구성해보려고 합니다. 운영 환경에서는 배치 서버를 단일 인스턴스로두고, API 서버와는 분리해서 구현할 예정입니다.
다만, 실제로 어떤 방식으로 구현하는 게 가장 적절할지 고민 중입니다.
예를 들어, 멀티모듈 구조로 API와 Batch를 분리하는 방식과 완전히 별도 프로젝트로 분리해서 독립 배포하는 방식 무엇이 효율적일지,
멀티모듈 구현해보는 것 자체에서 더 발전시킬 수 있는 것이 무엇이 있는지 궁금합니다!.
그리고 실제로도 배치서버를 API서버의 인스턴스와는 분리해서 새로운 인스턴스에 배포하는식으로 구현하는지 궁금합니다.
Q2. 현재 요구사항 중, 사용자의 리뷰를 수집하고, 리뷰에 맞는 특정 키워드(100 여개)를 llm을 통해서 추출을 할 예정입니다. 프롬프트 엔지니어링만을 이용해서 llm에게 전적으로 시킬 예정입니다. 혹시, vector db를 이용해서 정확도를 높이는 방법이 있을까요? RAG를 구현해본 적이 있는데 어떤 방향성으로 활용하면 좋을 지 궁금합니다 !
##💌 요청드리는 리뷰 방향
바쁘신 와중에 시간 내어 리뷰해주셔서 감사합니다.
편하게 피드백 부탁드립니다! 🥹