-
Notifications
You must be signed in to change notification settings - Fork 544
[SDK] Perf improvements with Promise.all #1648
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
Conversation
|
Most changes are published. Happy to close this |
Don't merge this
I'm aware this is a lot of changes. I will probably split it up into smaller PRs so that we have more control
Problem solved
We have a lot of Promise.all that are not calling the inner requests concurrently, rather sequencially.
Changes made
How to test
Self note: The following Pormise.all in this file can be improved however it seems too complex dont wanna touch that rn