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

Improve query speed #314

Closed
wngus4296 opened this issue Aug 6, 2022 · 5 comments
Closed

Improve query speed #314

wngus4296 opened this issue Aug 6, 2022 · 5 comments
Assignees
Labels

Comments

@wngus4296
Copy link
Member

  • 여러개의 쿼리를 실행 할 경우 속도를 개선합니다.
@wngus4296 wngus4296 self-assigned this Aug 6, 2022
@seoljiwon
Copy link
Contributor

seoljiwon commented Aug 7, 2022

이거 정확히 무슨 의미인지 궁금합니다!
(혹시 #301 과 같은 의미라면 close as not planned로 닫고 이슈 같이 해결해봐요
개발하면서 같이 진행할 수 있는 거면 아예 refactor하면서 신기능 개발해도 좋을 것 같아용!)

@wngus4296
Copy link
Member Author

wngus4296 commented Aug 8, 2022

이거 이번에 제 pr 보시면

const [invisibleUserList, relationReviewTagList] = await Promise.all([
      blockDB.getInvisibleUserListByUserId(client, req.user.id),
      relationReviewTagDB.getRelationReviewTagList(client),
    ]);

이런식으로 꼭 위에 쿼리 결과물이 있어야 돌아가는 쿼리가 아니면 한 번에 돌리려구요!

https://code-masterjung.tistory.com/91
여기 한 번 확인해주시면 될 듯 합니당

@seoljiwon
Copy link
Contributor

아하아하 안 그래도 진짜 똑같은 블로그 봤었어요 ㅋㅋㅋ 덕분에 공부했습니다
일단 DB단에서 최대한 join해서 가져와보고(#301 내용) 그래도 따로 가져와야 하는 게 있으면 이렇게 하시죵!

@wngus4296
Copy link
Member Author

아하아하 안 그래도 진짜 똑같은 블로그 봤었어요 ㅋㅋㅋ 덕분에 공부했습니다 일단 DB단에서 최대한 join해서 가져와보고(#301 내용) 그래도 따로 가져와야 하는 게 있으면 이렇게 하시죵!

ㅋㅋㅋㅋ좋습니다!! 🔥

@wngus4296
Copy link
Member Author

해당 이슈는 일단 보류하고 나머지 작업한 후에 진행하겠습니다~!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants