kevinrr888 commented on code in PR #4077: URL: https://github.com/apache/accumulo/pull/4077#discussion_r1431845210
########## core/src/main/java/org/apache/accumulo/core/client/admin/NamespaceOperations.java: ########## @@ -202,9 +203,9 @@ default Iterable<Entry<String,String>> getProperties(String namespace) } /** - * Gets a merged view of the properties of a namespace, which are inherited by tables in this - * namespace. Note that recently changed properties may not be available immediately. This new - * method returns a Map instead of an Iterable. + * Gets a merged view of the properties of a namespace from its parent configuration. These + * properties are inherited by tables in this namespace. Note that recently changed properties may + * not be available immediately. This new method returns a Map instead of an Iterable. Review Comment: Yes I agree. I found it strange that "new" was included. It was already present and I didn't want to make any unwanted changes. I'll remove it -- 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]
