[GitHub] [bookkeeper] horizonzy commented on a diff in pull request #3889: [improve] [client] Group read/write response when jumping to the ordered executor

["horizonzy (via GitHub)" <gi...@apache.org>]

horizonzy commented on code in PR #3889:
URL: https://github.com/apache/bookkeeper/pull/3889#discussion_r1150951754


##########
bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java:
##########
@@ -173,6 +178,7 @@ public class PerChannelBookieClient extends ChannelInboundHandlerAdapter {
                         BKException.Code.DuplicateEntryIdException,
                         BKException.Code.WriteOnReadOnlyBookieException));
     private static final int DEFAULT_HIGH_PRIORITY_VALUE = 100; // We may add finer grained priority later.
+    private static final int DEFAULT_CAPACITY = 1_000;

Review Comment:
   DEFAULT_CAPACITY > BATCH_RESPONSE_SIZE



-- 
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@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org