eolivelli commented on a change in pull request #2494:
URL: https://github.com/apache/bookkeeper/pull/2494#discussion_r531485307



##########
File path: 
bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerHandle.java
##########
@@ -173,6 +184,33 @@
     final Counter lacUpdateMissesCounter;
     private final OpStatsLogger clientChannelWriteWaitStats;
 
+    /**
+     * Read ahead related parameters
+     */
+    // Number of read ahead entries allowed in cache.
+    private final int maxNumReadAheadEntries = 0;

Review comment:
       can we group all of these fields into a separate datastructure ?
   all of these fields are useful only in case of read ahead feature enabled




----------------------------------------------------------------
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:
[email protected]


Reply via email to