dlmarion commented on code in PR #5627:
URL: https://github.com/apache/accumulo/pull/5627#discussion_r2138903620
##########
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:
probably need to add this to the fixed set of properties as we likely don't
want this changed at runtime. Wondering if we need to add it to the instance
properties to make sure it's the same across the cluster.
--
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]
