keith-turner commented on code in PR #5627:
URL: https://github.com/apache/accumulo/pull/5627#discussion_r2140406604
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -335,6 +335,9 @@ public enum Property {
+ " was changed and it now can accept multiple class names. The
metrics spi was introduced in 2.1.3,"
+ " the deprecated factory is
org.apache.accumulo.core.metrics.MeterRegistryFactory.",
"2.1.0"),
+
GENERAL_SERVER_ITERATOR_OPTIONS_COMPRESSION_ALGO("general.server.iter.opts.compression",
"none",
Review Comment:
seems ok for this to change at runtime because the compression is type is
included in the map sent to the tservers. So that is nice because it means the
tservers do not need to be in sync for that prop.
--
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]
