[GitHub] [accumulo] cshannon commented on a diff in pull request #2986: Re-add getProperties() to InstanceOperations

[GitBox <gi...@apache.org>]

cshannon commented on code in PR #2986:
URL: https://github.com/apache/accumulo/pull/2986#discussion_r984517348


##########
core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperations.java:
##########
@@ -93,6 +93,15 @@ void modifyProperties(Consumer<Map<String,String>> mapMutator) throws AccumuloEx
    */
   Map<String,String> getSystemConfiguration() throws AccumuloException, AccumuloSecurityException;
 
+  /**
+   * Retrieves the stored System properties from zookeeper. This method is different from
+   * {@link #getSystemConfiguration()} as it will only return the stored properties in zookeeper and
+   * not properties from accumulo.properties or default values..
+   *
+   * @return a map of stored system properties set in zookeeper
+   */
+  Map<String,String> getStoredProperties() throws AccumuloException, AccumuloSecurityException;

Review Comment:
   Sounds good, we can go back to the old name. I would only want to change it if there is agreement which there is not.



-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org