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/08/25 08:37:19 UTC

[GitHub] [pulsar] lhotari commented on pull request #14488: [improve][broker] refactor ManagedLedger cacheEvictionTask implement

lhotari commented on PR #14488:
URL: https://github.com/apache/pulsar/pull/14488#issuecomment-1226956377

   > > It would be good to fix #16054 before changing the cache.
   > 
   > @lhotari This pull request just refactors the eviction from an `eviction and sleep` loop to a regularly scheduled task and does not change any logic of the eviction task. IMO, this PR does not relate to #16054
   
   @aloyszhang I was just trying to get the most critical problem of the broker cache (#16054) fixed before we do any other changes. I have now pushed a fix in #17273 . Please review since it should make a difference for heavy users of Pulsar.


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