rdhabalia commented on a change in pull request #7910:
URL: https://github.com/apache/pulsar/pull/7910#discussion_r478848714
##########
File path: conf/broker.conf
##########
@@ -624,6 +624,9 @@ bookkeeperClientTimeoutInSeconds=30
# Using a value of 0, is disabling the speculative reads
bookkeeperClientSpeculativeReadTimeoutInMillis=0
+# Number of channels per bookie
+bookkeeperNumberOfChannelsPerBookie=16
Review comment:
right now, hardcoded value is `16` and it was configured > 1 due to per
bookie cnx bottleneck. So, changing it to 1 will create an issue for existing
system if they will try to use upgrade release without knowledge of this
configuration. However, recently while doing testing on high throughput
bookies, we figured out that 16 bookie channel was the bottleneck and we need
to make it configurable to tune it based on number topics and per bookie
traffic.
----------------------------------------------------------------
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:
us...@infra.apache.org
