Skip to content

Reverted PR #393 - #411

Merged
Gsantomaggio merged 2 commits into
mainfrom
revert-panic-chunk-dispatching
Jul 3, 2025
Merged

Gsantomaggio merged 2 commits into
mainfrom
revert-panic-chunk-dispatching

Conversation

@hiimjako

@hiimjako hiimjako commented Jul 3, 2025

Copy link
Copy Markdown
Collaborator

Reverted PR #393, since it was causing possible data loss in case of slow consumer handlers.
With a normal use it is not triggered, but under high pressure the client could discard messages without any notification in info logs.

@hiimjako
hiimjako requested a review from Gsantomaggio July 3, 2025 15:01
@hiimjako hiimjako self-assigned this Jul 3, 2025
@hiimjako hiimjako added the bug Something isn't working label Jul 3, 2025
@Gsantomaggio
Gsantomaggio merged commit 27bdca5 into main Jul 3, 2025
@Gsantomaggio Gsantomaggio added this to the 1.5.8 milestone Jul 3, 2025
@Gsantomaggio
Gsantomaggio deleted the revert-panic-chunk-dispatching branch July 3, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants