albertogpz commented on a change in pull request #7424:
URL: https://github.com/apache/geode/pull/7424#discussion_r821551163
##########
File path:
geode-wan/src/main/java/org/apache/geode/cache/wan/internal/WanCopyRegionFunctionService.java
##########
@@ -43,7 +45,9 @@
public boolean init(Cache cache) {
String WAN_COPY_REGION_FUNCTION_EXECUTION_PROCESSOR_THREAD_PREFIX =
"WAN Copy Region Function Execution Processor";
- int WAN_COPY_REGION_FUNCTION_MAX_CONCURRENT_THREADS = 10;
+ int WAN_COPY_REGION_FUNCTION_MAX_CONCURRENT_THREADS = SystemProperty
+ .getProductIntegerProperty(
+
SystemPropertyHelper.WAN_COPY_REGION_FUNCTION_MAX_CONCURRENT_THREADS, 10);
Review comment:
I have added a test case to make sure that when the maximum limit is
reached the new command will wait for a thread to be free.
I am not sure if it is really necessary or is it worth to add a cap for max
threads.
--
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
