You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/08/08 16:34:24 UTC

[GitHub] [pulsar] merlimat commented on a change in pull request #7765: make orderedExecutor threads number configurable

merlimat commented on a change in pull request #7765:
URL: https://github.com/apache/pulsar/pull/7765#discussion_r467481604



##########
File path: conf/broker.conf
##########
@@ -58,6 +58,10 @@ advertisedAddress=
 # Number of threads to use for Netty IO. Default is set to 2 * Runtime.getRuntime().availableProcessors()
 numIOThreads=
 
+# Number of threads to use for ordered executor. The ordered executor is used to operate with zookeeper,
+# such as init zookeeper client, get namespace policies from zookeeper etc. It also used to split bundle. Default is 8
+numOrderedExecutorThreads=

Review comment:
       If we have a fixed default, we should have the value here, unless the number is computed based on the available processors (which is in most cases a better approach)
   
   ```suggestion
   numOrderedExecutorThreads=8
   ```




----------------------------------------------------------------
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:
users@infra.apache.org