[GitHub] [geode] nabarunnag 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>]

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



##########
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:
       @agingade we created this PR after receiving feedback from customers and support. This is one of the requested features from the field and these are the numbers we have received from support.
   We are also aligning these numbers with MAX_FE_THREADS, which was changed to cpu*32 to tune the internal tests.
   
   This can be a temporary band-aid till we start on GEODE-8357, which has more extensive changes and we can involve the community feedback.




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