dlmarion commented on code in PR #5749:
URL: https://github.com/apache/accumulo/pull/5749#discussion_r2242778944
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -1668,6 +1668,13 @@ public static boolean isValidZooPropertyKey(String key) {
||
key.equals(Property.GENERAL_FILE_NAME_ALLOCATION_BATCH_SIZE_MAX.getKey());
}
+ public static boolean isValidResourceGroupPropertyKey(String property) {
+ return property.startsWith(Property.GENERAL_PREFIX.getKey())
Review Comment:
I didn't add that up to now because it seemed like some of the RPC
properties might be instance level. For example, if different servers have
non-overlapping sets of RPC_SSL_CIPHER_SUITES, then they won't be able to
communicate. I'm happy to modify this set though.
--
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
