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/02/01 02:51:02 UTC

[GitHub] [pulsar] sijie commented on pull request #9292: [pulsar-broker] Support large number of unack message store for cursor recovery

sijie commented on pull request #9292:
URL: https://github.com/apache/pulsar/pull/9292#issuecomment-770528282


   > Yes, I shared my point of view in email regarding having unack message management. However, regardless abstraction changes, I think we can proceed with this PR as this can be used with existing default unack message implementation.
   
   I would like to have a consensus on providing an abstraction before merging this implementation. Otherwise, it becomes really hard to allow people to implement other algorithms. 


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