-
Notifications
You must be signed in to change notification settings - Fork 37
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
[ZoomAccounts] Fix N+1 issue in availability field #18
Comments
Can i work on this @hereisnaman ? |
@coderrsid sure, I am assigning it to you. Let me know if you need any help. |
Although it'll be a big implementation. Sure i'll see what i can do. |
@coderrsid query FetchZoomAccounts {
ZoomAccounts(availableDuring: ...) {
id
availability(dateTimeRange: ...)
}
} The number of sql queries will be: |
Hey naman, should i wait for the another similar issue for CalendarInvite as he also added dataloader package, so i don't want that package version conflicts happen?? |
That is a 1 line change, how hard will it be to resolve it even if that comes? |
To improve the performance of the query by reducing the no. of separate db queries. Need to batch the availability status check queries in zoomAccount.availability.
Problem: N+1
Solution: Dataloader
Learn: https://www.youtube.com/watch?v=OQTnXNCDywA
The text was updated successfully, but these errors were encountered: