Skip to content

Performance of the sendEach, sendEachAsync methods is too slow. #941

Closed

Description

@honorhs

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.

Activity

  1. google-oss-bot commented on May 2, 2024

    @google-oss-bot

    I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.

  2. emindeniz99 commented on May 6, 2024

    @emindeniz99

    same issues: ( and no solution yet ) :) I hope a solution before 20 june 2024 because it is the deadline for old api removal. The solution should be implemented already because developers needs sometimes to implement to their systems. There is less than 2 months.

    #834

    #849

  3. raychongtk commented on May 9, 2024

    @raychongtk

    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.

  4. jwagun commented on Jun 18, 2024

    @jwagun

    Any update on this performance issue?

  5. Kyupile commented on Jun 18, 2024

    @Kyupile

    I'm trying to apply new apis but tt has absolutely serious problem of my projects.
    Compare to legacy, it costs too much seconds to get a response.
    Please reply any solution except using topic.

  6. georgeaniljohn commented on Jul 2, 2024

    @georgeaniljohn

    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.

  7. honorhs commented on Jul 8, 2024

    @honorhs
    Author

    Do you have any update on this? @lahirumaramba @jonathanedey

  8. moda666 commented on Aug 22, 2024

    @moda666

    Is this issue resolved?

  9. xiaolongbao321 commented on Sep 14, 2024

    @xiaolongbao321

    Is this issue resolved?

  10. Mzakimohammed commented on Sep 20, 2024

    @Mzakimohammed

    👍🤗

  11. jonathanedey commented on Oct 9, 2024

    @jonathanedey
    Contributor

    HTTP/2 support has now been added with the release of v9.4.0.
    See our wiki for more information on this change.

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

Metadata

Metadata

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions