berg223 commented on issue #23503: URL: https://github.com/apache/pulsar/issues/23503#issuecomment-2933685462
I have a preview on https://github.com/lhotari/pulsar/pull/209 . I have simplely splitted the improvement into two parts: 1. Strategy about when to insert entry into cache. We will insert entry into cache when expectedReadCountVal > 0 2. Strategy about when to remove entry from cache. We will remove entry from cache when the cache is full and expectedReadCountVal < 1 or expired. This is an awesome improvement. The strategy about insert is reactive, however it maybe too simple compared with `checkCursorsToCacheEntries`. Maybe we should consider compatibility about the insert strategy. We can make the origin insert strategy reactive by update `cacheReadEntry` filed of ManagedCursorImpl every time cursor updates position or accumulative update. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
