rdhabalia commented on a change in pull request #627: Reduce thread executor pool sizes when running unit tests URL: https://github.com/apache/incubator-pulsar/pull/627#discussion_r130997761
########## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java ########## @@ -78,6 +78,16 @@ * Main class for Pulsar broker service */ public class PulsarService implements AutoCloseable { + + private static final int EXECUTOR_THREAD_COUNT = Integer + .parseInt(System.getProperty("pulsar.executor-thread-count", "20")); Review comment: instead of reading from `System-property` what if we can make it configurable via `ServiceConfiguration` so, broker can be started with configured number of threads which may depend on host hardware or resource usage.? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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 With regards, Apache Git Services