ctubbsii commented on code in PR #3161:
URL: https://github.com/apache/accumulo/pull/3161#discussion_r1072728683
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -348,6 +348,16 @@ public enum Property {
+ "indefinitely. Default is 0 to block indefinitely. Only valid when
tserver available "
+ "threshold is set greater than 0. Added with version 1.10",
"1.10.0"),
+ MANAGER_LOW_MEM_DETECTOR_ACTIVE("manager.low.mem.detector.active", "false",
PropertyType.BOOLEAN,
+ "By default the LowMemoryDetector is passive, it just logs that memory
is low. Setting this to true"
+ + " may change the behavior of server components that check the
LowMemoryDetector state",
+ "3.0.0"),
+ MANAGER_LOW_MEM_DETECTOR_INTERVAL("manager.low.mem.detector.interval", "5s",
+ PropertyType.TIMEDURATION, "The number of milliseconds between low
memory checks", "3.0.0"),
+ MANAGER_LOW_MEM_DETECTOR_THRESHOLD("manager.low.mem.detector.treshold",
"0.05",
+ PropertyType.FRACTION,
+ "The LowMemoryDetector will report when free memory drops below this
percentage of total memory",
Review Comment:
Is there a reason we'd want this to be set differently on different server
types? We could just have one general property for these, instead of equivalent
properties for each server.
--
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]
