-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[fix] [ml] fix key_shared mode delivery are order out order after a consumers reconnection #23803
base: master
Are you sure you want to change the base?
[fix] [ml] fix key_shared mode delivery are order out order after a consumers reconnection #23803
Conversation
…onsumers reconnection
@poorbarcode Please add the following content to your PR description and select a checkbox:
|
Please update the PR title. It currently includes |
@poorbarcode PIP-282 (implementation PR #21953) was made to address known out-of-order issues in the Key_Shared implementation. PIP-282 was targeted for Pulsar 4.0 and later replaced with PIP-379 implementation. It feels that major changes to the Pulsar 3.0 Key_Shared implementation should be made carefully due to the high chance of changes causing other regressions. |
…ement (apache#23600) Co-authored-by: Lari Hotari <[email protected]>
…t-of-order, which may cause a delivery stuck (apache#23802)
… to reduce GC pressure (apache#23780)
…atibilityDifferentBrokersForNonPersistentTopic (apache#23666)
…tCnt while building consumer. (apache#23804)
Co-authored-by: houbonan <[email protected]>
…r Functions&Sources (apache#23793)
…amp (apache#22792) Co-authored-by: Lari Hotari <[email protected]>
…s when sending of message fails (apache#23824)
…pache#23832) Signed-off-by: Zixuan Liu <[email protected]>
…n if one fails (apache#23842) Signed-off-by: Zixuan Liu <[email protected]>
Signed-off-by: Zixuan Liu <[email protected]>
…ad in cancelPendingRead (apache#23384)" (apache#23855)
…ing and reconnection (apache#23853)
… config unackedRangesOpenCacheSetEnabled (apache#23759)
Motivation
Please review #23802 first, after the PR merged, we start the current PR.
The case that #23802 described may also lead to delivery out of order.
TODO explains more details.
Modifications
Documentation
doc
doc-required
doc-not-needed
doc-complete
Matching PR in forked repository
PR in forked repository: x