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

[feat] : 전체 Gallery 조회 기능 추가 #381

Merged
merged 6 commits into from
Mar 3, 2024
Merged

[feat] : 전체 Gallery 조회 기능 추가 #381

merged 6 commits into from
Mar 3, 2024

Conversation

devxb
Copy link
Member

@devxb devxb commented Mar 1, 2024

어떤 기능을 개발했나요?

전체 Gallery 조회 기능을 추가했습니다.

고민포인트

  • 조회 성능 향상용으로 커버링인덱스 부착 or 안부착? (지금은 부착)
  • update, bookmark는 카디널리티가 낮고 변경이 잦은데 index 부착이 맞는 선택인가? (지금은 정렬 + 커버링 인덱스 태우기용으로 index부착)
  • Pagination 보다 좋은 방법이 있을지?
    무한스크롤 쿼리 구현할때, 마지막 gallery id 찍고 다음꺼부터 내려주는식으로 하려고 했는데, 지금 update, bookmark 정렬 필터가 추가로 들어가면서 이 방식으로 구현이 힘듬 (update, bookmark로 정렬하는순간 마지막 gallery id가 필요 없기때문)
    그래서 Pagination 방식으로 구현했음.

어떻게 해결했나요?

  • gallery 테이블에 커버링 인덱스 추가
  • Pagination 방식으로 구현

이슈 넘버

@devxb devxb self-assigned this Mar 1, 2024
@devxb devxb changed the title feat: 전체 Gallery 조회 기능 추가 [feat] : 전체 Gallery 조회 기능 추가 Mar 1, 2024
Copy link
Member

@dojinyou dojinyou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

인메모리로 캐시 처리하고 업데이트 주기를 만드는 게 최선으로 보임

@devxb devxb closed this Mar 1, 2024
@devxb devxb reopened this Mar 1, 2024
Copy link

sonarqubecloud bot commented Mar 1, 2024

@devxb devxb merged commit f3cb875 into main Mar 3, 2024
4 checks passed
@devxb devxb deleted the devxb/iss-#362 branch March 3, 2024 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] : 전체 Gallery 조회 API 개발
2 participants