[GitHub] [geode] agingade commented on a change in pull request #5862: GEODE-8799: Increase defaults for OperationExecutors.MAX_THREADS and ClusterOperationExecutors.MAX_PR_THREADS

[GitBox <gi...@apache.org>]

agingade commented on a change in pull request #5862:
URL: https://github.com/apache/geode/pull/5862#discussion_r545321822



##########
File path: geode-core/src/main/java/org/apache/geode/distributed/internal/OperationExecutors.java
##########
@@ -26,7 +26,7 @@
  */
 public interface OperationExecutors {
   int MAX_THREADS =
-      Integer.getInteger("DistributionManager.MAX_THREADS", 100);
+      Integer.getInteger("DistributionManager.MAX_THREADS", 300);

Review comment:
       There is another ticket to address this issue: GEODE-8357
   This change has an impact on overall system requirement and performance; it will be nice to send a proposal to developer community. This helps to get feedback from engineers who had encountered issue with this setting and suggested the recommendation to the users (support teams); there was conversation about changing this to big number like 1000. 
   It will be good to come up with a right number based on previous experience in setting this number.
   
   We also need to think about setting the system properties automatically; some kind of auto-tuning service in the product that will adopt/changes the value based on the resource availability and the system load.
   




----------------------------------------------------------------
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