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:
[email protected]
