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

페이지네이션 처리 시 최대 다음 페이지 수를 5개로 제한 #902

Open
wants to merge 5 commits into
base: dev/be
Choose a base branch
from

Conversation

jminkkk
Copy link
Contributor

@jminkkk jminkkk commented Nov 8, 2024

⚡️ 관련 이슈

close #899

📍주요 변경 사항

  1. 목록 조회에서 페이지네이션이 들어갈 경우, Page 반환타입은 전체 페이지와 요소 갯수를 반환합니다.
    그 과정에서 전체 로우를 풀스캔하기 때문에 비효율적입니다.

(조회 조건에 해당되는 컬럼이 100만건이라면 100만 로우를 전체 카운팅)

현재 페이지네이션 바가 5개까지 나타나기 때문에 전체 로우를 카운팅하지 않고 다음 5개 페이지까지가 있는지 반환합니다.

예) 
조건에 맞는 전체 페이지가 10개인 경우: nextPages는 5개 
조건에 맞는 전체 페이지가 1개인 경우: nextPages는 1개 
  1. 내가 좋아요한 템플릿 목록 조회 API에서 동일한 페이지네이션 바가 사용되기 때문에 템플릿 목록 조회 API와 동일하게 5개까지만 반환하는 처리를 하였습니다.
    그 과정에서 내가 좋아요한 "템플릿" 목록 조회 레포지토리 메서드를 template 패키지로 이동하였습니다. (이전 담당자와 상의 완, 참고바람~~)

🎸기타

관련 논의 디스커션 참고: 템플릿 목록 응답의 전체 템플릿 수의 필요성에 대해 이야기해봐요

+) Repository 단 테스트 시에 관련 Config 들을 Import해야하는데 @TemplateSearchRepositoryTest와 @RepositoryTest@import가 중복으로 수정되네욥ㅠ 이번 PR에서 변경사항이 많이 잡힐 것 같아 따로 PR 올릴게욥

🍗 PR 첫 리뷰 마감 기한

11/11 18:00

@jminkkk jminkkk added refactor 요구사항이 바뀌지 않은 변경사항 BE 백엔드 labels Nov 8, 2024
@jminkkk jminkkk added this to the 7차 스프린트 💭 milestone Nov 8, 2024
@jminkkk jminkkk self-assigned this Nov 8, 2024
@jminkkk jminkkk changed the base branch from main to dev/be November 8, 2024 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 refactor 요구사항이 바뀌지 않은 변경사항
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 검색 기능 개선
1 participant