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 2022/07/29 09:52:02 UTC

[GitHub] [pulsar] BewareMyPower commented on pull request #15474: [fix][broker] Fix cannot cleanup expired ledger by trim ledgers

BewareMyPower commented on PR #15474:
URL: https://github.com/apache/pulsar/pull/15474#issuecomment-1199087461

   I think we still need to revert #14672 first (#16806). Because this PR just checks whether any cursor has been updated in the the periodically `BrokerService#checkConsumedLedgers` call based on #16806, which restores the original behavior (same with branch-2.10 and earlier) that a new ledger will be created after closing the current ledger.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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