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

[Academy schedules] 학원 Location 관련 목록 조회 쿼리 개선 #121

Merged
merged 11 commits into from
Dec 9, 2023

Conversation

byeolhaha
Copy link
Member

구현

  • 학원 Location 관련 목록 조회 쿼리 개선
  • no offset 방식으로 채택하려고 했으나 기존보다 더 성능이 나빠져서 200개 데이터를 가지고 no offset을 하는 것이 오히려 비효율적인 것 같다는 생각이 들었습니다.
  • 그래서 기존의 order by에서 학원 이름으로 정렬을 했는데 학원 이름이 인덱스가 아니어서 r-index를 사용함에도 쿼리 성능이 나오지 않아 이 부분을 삭제하게 되었습니다.

@byeolhaha byeolhaha changed the base branch from main to develop December 8, 2023 17:22
@byeolhaha byeolhaha requested review from onetuks and wonu606 December 8, 2023 17:22
@byeolhaha byeolhaha self-assigned this Dec 8, 2023
wonu606
wonu606 previously approved these changes Dec 8, 2023
@byeolhaha byeolhaha merged commit 34d637c into develop Dec 9, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants