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


##########
core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java:
##########
@@ -643,6 +664,9 @@ default Iterable<Entry<String,String>> getProperties(String 
tableName)
   Map<String,String> getConfiguration(String tableName)
       throws AccumuloException, TableNotFoundException;
 
+  Map<String,String> getTableSpecificConfiguration(String tableName)

Review Comment:
   The difference is that getConfiguration() returns all properties (system, 
table, etc) which is a problem because when updating the properties we only 
want to properties that can be mutated on a table. If you use the original 
method you get back a much larger map of properties and when you send it back 
the validation fails due to the invalid properties.



-- 
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]

Reply via email to