keith-turner commented on code in PR #5743:
URL: https://github.com/apache/accumulo/pull/5743#discussion_r2219798101
##########
server/base/src/main/java/org/apache/accumulo/server/util/PropUtil.java:
##########
@@ -74,7 +77,28 @@ protected static void validateProperties(final ServerContext
context,
"Unable to resolve classloader for context: " + prop.getValue());
}
}
+
+ if (prop.getKey().equals(Property.TABLE_ERASURE_CODE_POLICY.getKey())) {
Review Comment:
Also not sure how the property inheritance hierarchy would factor into
validation of multiple props when setting a prop. Would need to render the
projected merged view of all the prop stores plus the changes to do this which
would further complicate making the change. Like if replication is set at the
system level and we are trying to set ec at the table level.
--
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...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
