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

#293 [fix] 디자이너 메인뷰 페이징 정렬, total 계산 오류 해결 #294

Merged
merged 4 commits into from
Nov 12, 2024

Conversation

hellozo0
Copy link
Member

@hellozo0 hellozo0 commented Nov 12, 2024

관련 이슈번호

해결하는 데 얼마나 걸렸나요? (예상 작업 시간 / 실제 작업 시간)

  • 15m/30m

해결하려는 문제가 무엇인가요?

  • 정렬이 생성 날짜 별로 _ 최근것이 아니였다 => 정렬 기준 변경
  • total 계산 오류 => non Expired 전체 계산히 total도 다시 계산 하도록 변경

추가 개선 사항

  • 어쩔 수 없이 expired 계산 처리후에 값을 가져와야한다 , 또한 offset방식으로 페이징 처리를 하고 있기 때문에 클라이언트에서 size를 넘어가게 되면 api 요청을 한번 더 하게 된다. 그러므로 total 개수 계산은 매번 이뤄져야한다.
  • 그래서 만료되지 않는 total 요소의 개수를 쿼리를 통해 가져올 수 있도록 했다.
  • 인덱스를 생성하고, count h 대신에 count * 를 통해 B Tree가 요소 레이어까지 접근하지 않고 인덱스 개수만 조회해서 가져올 수 있도록.. 나름의 조회를 빠르게 할 수 있도록 했다.

Wanna Do

오프셋 방식 말고... 커서 방식... need... 클라이언트와 이야기를 나눠야한다
사실 다 뜯어 고쳐서 최적화하고 싶지만 배포되어 있는 서비스에 문제가 발생한거라 추후 수정해야할듯

@hellozo0 hellozo0 added the fix label Nov 12, 2024
@hellozo0 hellozo0 self-assigned this Nov 12, 2024
1. 인덱스 생성후 created at으로 쿼리 타기
2. count(h) 보다 count *가 조회성능이 더 빠르기 때문에  변경 ( B tree 요소까지 접근 안해도 인덱스만 조회하면 되서 빠른거다 )
@hellozo0 hellozo0 changed the base branch from main to develop November 12, 2024 14:42
@hellozo0 hellozo0 merged commit 4f934fa into develop Nov 12, 2024
1 check passed
@hellozo0 hellozo0 deleted the fix/#293 branch November 12, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 디자이너 메인뷰 페이징 오류
1 participant