ctubbsii commented on a change in pull request #2122:
URL: https://github.com/apache/accumulo/pull/2122#discussion_r643281439
##########
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:
To be consistent with those, getTableConfiguration and
getNamespaceConfiguration might be most appropriate, but those are quite long,
so I'm not 100% sold on those.
--
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]
