-
Notifications
You must be signed in to change notification settings - Fork 285
Performance of the sendEach, sendEachAsync methods is too slow. #941
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
Comments
I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight. |
For large-scale systems, FCM is now way too expensive for deployment cost after deprecating Batch API. In our scenario, our push notifications are intense and traffic is huge and continuous. With the new API endpoint, we found that the performance is too bad unless we deploy many machines to handle the traffic. For this issue, I would like to know if the FCM team has any suggestions or not because not every scenario is suitable to use Topic. |
Any update on this performance issue? |
I'm trying to apply new apis but tt has absolutely serious problem of my projects. |
We also faced similar situation. We then moved away from firebase-admin-java SDK and have our own implementation where we use HTTP/2 and multiplexing. For now this works for us. |
Do you have any update on this? @lahirumaramba @jonathanedey |
Is this issue resolved? |
1 similar comment
Is this issue resolved? |
👍🤗 |
After updating to version 9.2.0 of the firebase-admin-java, I experienced performance issues and had to rollback to a previous version.
The performance of sendEachAsync is several to tens of times slower compared to sendAllAsync.
I don't understand why there is a decision to not provide a batch API anymore, forcing a migration to sendEach or sendEachAsync without any alternatives.
Shouldn't the SDK be provided at a reasonable level with equivalent performance?
Moreover, it does not even use HTTP/2 connections.
Why are all the performance and resource cost issues caused by SDK v9.2.0 passed onto the users?
I believe the batch API should be maintained until an alternative (API) that can match the performance of the existing batch API is developed.
I would like to hear Firebase position on this matter.
The text was updated successfully, but these errors were encountered: