milleruntime commented on a change in pull request #1651:
URL: https://github.com/apache/accumulo/pull/1651#discussion_r516945149
##########
File path:
core/src/main/java/org/apache/accumulo/core/metadata/schema/Ample.java
##########
@@ -119,6 +119,15 @@ public static DataLevel of(TableId tableId) {
*/
TabletMetadata readTablet(KeyExtent extent, ColumnType... colsToFetch);
+ /**
+ * Entry point for reading multiple tablets' metadata. Generates a
TabletsMetadata builder object
+ * and assigns the AmpleImpl client to that builder object. This allows
readTablets() to be called
+ * from a ClientContext. Associated methods of the TabletsMetadata Builder
class are used to
+ * generate the metadata.
+ */
+
+ AmpleImpl.Builder readTablets();
Review comment:
This exposes the Impl class on the interface. This breaks the contract
of the interface.
----------------------------------------------------------------
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]
