kevinrr888 opened a new issue, #5695: URL: https://github.com/apache/accumulo/issues/5695
There are several prefixes not defined in `isValidZooPropertyKey`, which I'm not sure is by design. For example, should `isValidZooPropertyKey` include any of `RPC_PREFIX`, `INSTANCE_PREFIX`, `MONITOR_PREFIX`, `COMPACTOR_PREFIX`, etc? It seems like `FIXED_PROPERTIES` is meant to be a subset of those that are `isValidZooPropertyKey` (see descriptions for `isFixedZooPropertyKey` and `isValidZooPropertyKey`). However, there are several cases of a fixed property which would not be `isValidZooPropertyKey` (e.g., `COMPACTOR_PORTSEARCH` is in `FIXED_PROPERTIES` but would return false for `isValidZooPropertyKey`). Either 1) At least one description is incorrect and should be updated 2) `isValidZooPropertyKey` is missing some prefixes and should be updated 2 seems most likely _Originally posted by @kevinrr888 in https://github.com/apache/accumulo/pull/5632#pullrequestreview-2917620497_ -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
