lhotari commented on code in PR #24623:
URL: https://github.com/apache/pulsar/pull/24623#discussion_r2306478824


##########
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/Entry.java:
##########
@@ -67,6 +67,17 @@ public interface Entry {
      */
     boolean release();
 
+    /**
+     * Managed Ledger implementations of EntryImpl should implement this 
method to return the read count handler
+     * associated with the entry.
+     * This handler is used to track how many times the entry has been read 
and to manage
+     * the eviction of entries from the broker cache based on their expected 
read count.
+     * @return
+     */
+    default EntryReadCountHandler getReadCountHandler() {

Review Comment:
   I removed the duplicate `getReadCountHandler` from `RefCountedEntry` and 
moved `hasExpectedReads` to `Entry` interface to make it more consistent.



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

Reply via email to