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

FR: Async requests #104

Open
eranhirs opened this issue Dec 11, 2017 · 22 comments
Open

FR: Async requests #104

eranhirs opened this issue Dec 11, 2017 · 22 comments
Assignees

Comments

@eranhirs
Copy link

I am using firebase-admin on an asyncio project and when there are a lot of requests it slows down the whole API.

Surprisingly, I didn't see any other issues about this, I hope I am not duplicating.

ref = yield from db.reference
result = yield from ref.get()

Thanks

@hiranya911
Copy link
Contributor

Can this be not handled in the application code? Basically execute Admin SDK methods like ref.get() on separate threads.

@eranhirs
Copy link
Author

It won't scale as I would like and, to my opinion, complicates the code and is harder to track.

@hiranya911
Copy link
Contributor

Most of our users are on Python 2.7, so we cannot directly expose async coroutines from the SDK anytime soon. In the meantime you should be able to get around the scalability issues by using the run_in_executor() method of the asyncio event loop.

ref = db.reference(path) # Doesn't make RPC calls

executor = concurrent.futures.ThreadPoolExecutor(max_workers=8)
loop = asyncio.get_event_loop()
loop.run_in_executor(executor, ref.get) # Make RPC call on a separate thread

@eranhirs
Copy link
Author

eranhirs commented Dec 13, 2017

In case anyone runs into the same problem -
Eventually, I am using the REST API.
Calling it asynchronously with aiohttp.

For auth I am using the database secret which is DEPRECATED, so it's not ideal, but it works for now.

@Natim
Copy link

Natim commented Mar 5, 2019

@eranhirs would you feel like starting a async-firebase project that we could contribute to?

@hyperknot
Copy link

I wanted to report that having async support for an auth SDK is very important. This is exactly the area where async frameworks make a huge difference, where you do a client side request inside a server side request.

Please consider adding core support and examples for using with ASGI frameworks, like Starlette:
https://github.com/encode/starlette

@Lawouach
Copy link

I agree with the general feeling here. Not having async support is making this library a tad less useful nowadays. Not very future proof unfortunately.

@labroid
Copy link

labroid commented Jan 1, 2020

+1 on this. My project has a few hundred thousand items that need to be worked via Firebase, and ~1 second latency times make wait times intractable without async support

@labroid
Copy link

labroid commented Jan 1, 2020

Python 2.7 is no longer supported. Any progress here?

@hiranya911
Copy link
Contributor

This needs to be first addressed in other GCP Python libraries that we use as dependencies. More specifically, the google-auth library (which we use to make all authorized HTTP calls) needs to support it. From what I can see, they have plans to add async support starting from 2020, but there is no concrete timetable as of now:

googleapis/google-auth-library-python#321
googleapis/google-cloud-python#3103

I can only imagine that this will take a while considering the engineering effort that is involved. Pretty much all existing APIs have to be re-evaluated and redesigned for async support.

If google-auth takes a very long time to introduce async support, we can consider wrapping the existing google-auth APIs in async threads, so that at least firebase-admin can expose an async API to the developer. But even that is going to require a lot of work, based on some prototyping results I've managed to put together.

noahkw added a commit to noahkw/botw-bot that referenced this issue Feb 7, 2020
Make Firebase calls asynchronous by
running them in a ThreadPoolExecutor:
firebase/firebase-admin-python#104 (comment)

This touches a lot of code, because a new formatter was
introduced as well.
@kamyar
Copy link

kamyar commented Jun 3, 2020

Is there any development on this front?
Also can you please reopen this issue?
We have been using firestore and lack of async is a very big downside of firestore.
If async supported is not planned for google services, It would be nice if we know beforehand so we can plan our migration to services that support it. Lack of this feature is causing us both money and product quality tbh.

@hiranya911
Copy link
Contributor

There's been no development in this space afaik. We cannot support an async API until the core GCP Python libraries like google-auth provide async support. For Firestore you can file a feature request directly at https://github.com/googleapis/python-firestore

@kamyar
Copy link

kamyar commented Jun 3, 2020

It seems there is an open PR related to adding async support.

One of the contributors to the libraries just left a comment.

Would be great to hear about firebase-admin maintainers/contributors perspective on this. :)

Edit: I will look into the repo you linked and file a feature request maybe, thanks for the reply :)

@hiranya911
Copy link
Contributor

Good to know that python-api-core is in the process of adding async support. After that we need at least google-auth to adopt asyncio before we can provide async support (we'd very much like to do so btw). I see that you're already following their issue at googleapis/google-auth-library-python#321.

Firestore APIs are entirely developed in the python-firestore repo I linked above, and it's their call to support asyncio or not. But I think having async support python-api-core makes it feasible for them to support it.

@kamyar
Copy link

kamyar commented Jun 3, 2020

Great to hear these, thanks a lot @hiranya911 :)
I will look around in the python-firestore for the discussion, and maybe check if they have something planned. 🚀

@bradreardon
Copy link

It seems that an async-based auth flow was added to google-auth in September of last year: https://github.com/googleapis/google-auth-library-python/blob/master/CHANGELOG.md#1220-2020-09-28
googleapis/google-auth-library-python#612

Are there any plans to make this library async-compatible now that its dependencies seem to have done so?

@lahirumaramba
Copy link
Member

Good news! We are prioritizing this task and have plans to start the initial planning work over the coming months. It is still too early to provide a timeline, but I will reopen this issue so we can track the progress here. Thank you everyone for your patience!

@lahirumaramba lahirumaramba reopened this Apr 9, 2021
@miksa
Copy link

miksa commented Dec 20, 2021

Hi! Is there any update about this feature ? I saw a pull request to use new version of the google-auth api, would that fulfill prerequisites ?

@guneemwelloeux
Copy link

Hello! I know the last comment wasn't left here long ago, but as it didn't get any answer, I'm posting another one.
Has a timeline been agreed upon since April 2021, and if there is one, could you please let everyone know here?

Many thanks!

@lahirumaramba
Copy link
Member

Thank you again for your patience on this!
This is a high priority task for us and we should start seeing the initial changes sometime in Q2/Q3 2022. I can't promise you a timeline for this feature but rest assured that this is something on our immediate roadmap for this year and you will see updates in the coming months.

@jkyle109
Copy link
Contributor

jkyle109 commented Aug 25, 2022

Hey Folks, We just released v5.3.0 which added async support for the Cloud Firestore API. This is the first step into fulfilling this FR so please give it a try and let us know what you think!

@antont
Copy link

antont commented Feb 2, 2024

Seems to be working well, we haven't used this under heavy load yet, though.

I'm curious to hear how others are experiencing this, but AFAIK it's good and robust.

We mostly use firestore and do async gcloud storage uploads too, so things are pretty good.

Messages seems to be one thing missing? #639
That would be actually nice to have.

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

No branches or pull requests