ctubbsii opened a new issue, #5901: URL: https://github.com/apache/accumulo/issues/5901
The upgrade code changes in #5781 copies system properties into each existing namespace configuration, including the built-in namespace. Code in NamespaceConfiguration has a method `isIteratorOrConstraint` that is used with `Namespace.ACCUMULO` to prevent inheriting constraints or iterators from any system level configuration, so that users' attempts to configure system-wide table settings at the system level don't break the metadata tables. However, the upgrade code copies these to the namespace directly. So, any such properties that would have been filtered out from the parent config, are now permitted in the namespace config for the accumulo namespace. Two things should happen: 1. The NamespaceConfiguration code can be cleaned up, because it will no longer be necessary to filter out these configurations from the system configuration, since they are no longer allowed. 2. The upgrade code should skip over the same configurations when copying to the accumulo namespace that would have been filtered out by NamespaceConfiguration. -- 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
