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 2021/04/27 16:51:17 UTC

[GitHub] [pulsar] devinbost commented on pull request #10400: [Broker] Fix issue in reusing EntryBatchIndexesAcks instances

devinbost commented on pull request #10400:
URL: https://github.com/apache/pulsar/pull/10400#issuecomment-827757205


   @lhotari Good find! Thanks for pinging me on it.
   I think there's still another issue lurking that's causing the subscriptions to get stuck because we see the stuck subscriptions even when `acknowledgmentAtBatchIndexLevelEnabled` is disabled (which is the default.) However, when `acknowledgmentAtBatchIndexLevelEnabled` is true, this PR will prevent issues as you discovered. I hope we can merge this fix before many people start enabling this feature. 


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