foster33 commented on a change in pull request #2122:
URL: https://github.com/apache/accumulo/pull/2122#discussion_r644016098
##########
File path:
core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
##########
@@ -617,10 +617,28 @@ void removeProperty(String tableName, String property)
* recently changed properties may not be visible immediately.
* @throws TableNotFoundException
* if the table does not exist
+ * @deprecated since 2.1.0; use {@link #getPropertiesMap(String)} instead.
*/
+ @Deprecated(since = "2.1.0")
Iterable<Entry<String,String>> getProperties(String tableName)
throws AccumuloException, TableNotFoundException;
+ /**
+ * Gets properties of a table. This operation is asynchronous and eventually
consistent. It is not
+ * guaranteed that all tablets in a table will return the same values.
Within a few seconds
+ * without another change, all tablets in a table should be consistent. The
clone table feature
+ * can be used if consistency is required. This new method returns a Map
instead of an Iterable.
+ *
+ * @param tableName
+ * the name of the table
+ * @return all properties visible by this table (system and per-table
properties). Note that
+ * recently changed properties may not be visible immediately.
+ * @throws TableNotFoundException
+ * if the table does not exist
+ */
+ Map<String,String> getPropertiesMap(String tableName)
Review comment:
> Yeah those are long. The suggestion of `getConfiguration` is more
similar to the ones in InstanceOperations.
I have gone with `getConfiguration` . I think the consensus was this was the
most appropriate in this situation.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
