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

Optimizing Performance and Resource Utilization with Concurrent Async Execution ✈ #626

Merged
merged 4 commits into from
Nov 15, 2023

Conversation

sanjaiyan-dev
Copy link
Contributor

This PR enhances performance and optimizes hardware resource utilization by concurrently executing asynchronous functions using Promise.all.

@sanjaiyan-dev sanjaiyan-dev requested review from a team as code owners October 27, 2023 08:19
Copy link
Member

@kamilogorek kamilogorek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will throw on any promise rejecting, but because previous the behaviour was already not handling this correctly, I assume it's fine to change it like that.

@kamilogorek
Copy link
Member

Need some types updates.

@sanjaiyan-dev
Copy link
Contributor Author

Need some types updates.

@kamilogorek Is this correct now ? I hope so.

@sanjaiyan-dev
Copy link
Contributor Author

👀

@soedirgo
Copy link
Member

Looks good, thanks!

@soedirgo soedirgo merged commit 173235a into supabase:master Nov 15, 2023
4 checks 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.

3 participants