kohlmu-pivotal commented on code in PR #7424:
URL: https://github.com/apache/geode/pull/7424#discussion_r859137060
##########
geode-core/src/main/java/org/apache/geode/internal/lang/SystemPropertyHelper.java:
##########
@@ -113,6 +113,14 @@ public class SystemPropertyHelper {
*/
public static final String RE_AUTHENTICATE_WAIT_TIME =
"reauthenticate.wait.time";
+ /**
+ * Maximum number of concurrent executions in a server of wan-copy region
commands.
+ * Once the maximum number is reached, subsequent executions will be halted
until
+ * a thread for any of the ongoing executions is released.
+ */
+ public static final String WAN_COPY_REGION_MAX_CONCURRENT_THREADS =
+ "geode.wan.copy-region.max-threads";
Review Comment:
@pivotal-jbarrett can you please elude to what the inconsistencies are and
how you believe they should be resolved. (possibly even a link to a doc and
agreed naming approach) As I had previously asked @albertogpz to change it to
something more contextual and it seems believe it to be inconsistent.
In addition you should see that I had previously (on 8 Mar) asked yourself
and @upthewaterspout to comment on the system properties naming convention.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
