You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/08/20 13:10:25 UTC

[GitHub] [pulsar] massakam opened a new pull request #7861: [Issue 7554][broker] Split message ranges by ledger ID and store them in individualDeletedMessages

massakam opened a new pull request #7861:
URL: https://github.com/apache/pulsar/pull/7861


   Fixes #7554
   
   ### Motivation
   
   As mentioned in #7554, the class of `individualDeletedMessages` is different between Pulsar 2.3.2 (and earlier) and 2.4.0 (and later). This causes some of ranges contained in `individualDeletedMessages` to be lost when the version of Pulsar is upgraded, and a large number of messages that have already been acked can be redelivered to consumers.
   
   Also, even if the Pulsar version is 2.4.0 or later, the same phenomenon occurs when the value of `managedLedgerUnackedRangesOpenCacheSetEnabled` is switched from false to true.
   
   ### Modifications
   
   If the list of individually deleted message ranges loaded from ZK contains ranges that span different ledgers, split the ranges by ledger ID and store them in `individualDeletedMessages`.
   
   As a result, information about deleted message ranges is never lost and messages that have already been acked will not be redelivered.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] sijie merged pull request #7861: [Issue 7554][broker] Split message ranges by ledger ID and store them in individualDeletedMessages

Posted by GitBox <gi...@apache.org>.

sijie merged pull request #7861:
URL: https://github.com/apache/pulsar/pull/7861


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org