dschneider-pivotal commented on a change in pull request #7456:
URL: https://github.com/apache/geode/pull/7456#discussion_r832734304
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/cache/control/InternalResourceManager.java
##########
@@ -66,11 +66,15 @@
public class InternalResourceManager implements ResourceManager {
private static final Logger logger = LogService.getLogger();
- final int MAX_RESOURCE_MANAGER_EXE_THREADS =
- Integer.getInteger(GeodeGlossary.GEMFIRE_PREFIX +
"resource.manager.threads", 1);
+ private final int MAX_RESOURCE_MANAGER_EXE_THREADS =
+ Integer.getInteger(GeodeGlossary.GEMFIRE_PREFIX +
"resource.manager.threads", 2);
Review comment:
The poller thread used to create its own ScheduledPoolExecutor and now
it uses this existing one on the InternalResourceManager. I changed the default
number of threads from 1 to 2 to decrease the changes that the poller would be
stuck behind other operations that use this executor. We could go back to
having a dedicated executor.
--
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.
To unsubscribe, e-mail: notifications-unsubscr...@geode.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
