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.

Sign up for GitHub

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

Jump to bottom

bf: BB-419 notification: don't wait for delivery report #2421

Closed
wants to merge 3 commits into from
Closed

bf: BB-419 notification: don't wait for delivery report #2421

wants to merge 3 commits into from

Commits on Jun 14, 2023

  1. BB-419 adjust notification producer params 

    Adjust `batch.num.messages` and `queue.buffering.max.ms` values to
improve throughput a bit.

Note: those values have been applied in the field with success, they
may not be the best for every customer but they gave good result so
applying them on this basis.
    @jonathan-gramain
    jonathan-gramain committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    31c445e View commit details
    Browse the repository at this point in the history

  2. bf: BB-419 notification: don't wait for delivery report 

    Notification QueueProcessor doesn't wait for Kafka delivery reports
anymore, in order to speed up delivery of messages.

Also fix and improve logging related to processing errors and error
reporting from delivery reports.
    @jonathan-gramain
    jonathan-gramain committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    48552f6 View commit details
    Browse the repository at this point in the history

  3. BB-419 bump Backbeat version

    @jonathan-gramain
    jonathan-gramain committed Jun 14, 2023
    Configuration menu
    Copy the full SHA
    0d3323a View commit details
    Browse the repository at this point in the history