[
https://issues.apache.org/jira/browse/GEODE-8799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17251281#comment-17251281
]
ASF GitHub Bot commented on GEODE-8799:
---------------------------------------
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:
us...@infra.apache.org
> Increase default OperationExecutors#MAX_THREADS and MAX_PR_THREADS
> ------------------------------------------------------------------
>
> Key: GEODE-8799
> URL: https://issues.apache.org/jira/browse/GEODE-8799
> Project: Geode
> Issue Type: Improvement
> Affects Versions: 1.14.0
> Reporter: Donal Evans
> Assignee: Donal Evans
> Priority: Major
> Labels: pull-request-available
>
> Similar to GEODE-8780, the default maximum number of pooled message processor
> and partitioned region message processor threads are low enough that they can
> cause performance degradation and should be increased.
> The current default value for {{MAX_THREADS}} is 100, and the current default
> value for {{MAX_PR_THREADS}} is
> {{Math.max(Runtime.getRuntime().availableProcessors() * 4, 16)}}
> The proposed new values are 300 and
> {{Math.max(Runtime.getRuntime().availableProcessors() * 32, 200)}}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
