-
Notifications
You must be signed in to change notification settings - Fork 47
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
Async #69
Comments
Hello @thomaspaulgrey! Would you be able to provide any details about your use case for async support? This would help us with prioritizing feature work. |
I'm using the Squrae SDK in my backend and async requests makes it very easy to perform other tasks while waiting for the response to a request to Square. If the "other tasks" are also io/network bound (e.g. more requests to Square 😄), I'm likely also running them in an async way, such that I can combine the waiting time (as such, the time I wait is approx. equal to the time the slowest task takes). This is naturally much faster than doing all the waiting sequentially. And yes, much of that could be done using threads, but there are various disadvantages (especially for i/o bound operations) to using threading over asyncio. See e.g. this great comment. See also these fastapi docs providing a detailed, yet simple and intuitive motivation for using async requests. |
Hi there, I'm sorry to report that at this time we will not be adding 'async' functionality to the SDK. I will leave this issue open as a data point for when we go to make feature improvements to our SDKs though. For now it will be marked as |
Hi there!
I was wondering whether or not this library will support async anytime soon
The text was updated successfully, but these errors were encountered: