dlmarion commented on code in PR #2792:
URL: https://github.com/apache/accumulo/pull/2792#discussion_r973226980
##########
server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectionEnvironment.java:
##########
@@ -72,6 +72,17 @@ public interface GarbageCollectionEnvironment {
*/
Stream<Reference> getReferences();
+ /**
+ * Return a list of all TableIDs in the
+ * {@link org.apache.accumulo.core.metadata.schema.Ample.DataLevel} for
which we are considering
+ * deletes. When operating on DataLevel.USER this will return all user table
ids. When operating
+ * on DataLevel.METADATA this will return the table id for the
accumulo.metadata table. When
+ * operating on DataLevel.ROOT this will return the table id for the
accumulo.root table.
+ *
+ * @return The table ids
+ */
+ Set<TableId> getCandidateTableIDs();
Review Comment:
Moved javadoc around as suggested in
1f8d37305789fa1540158e32b54da86a7af3ae87
##########
server/gc/src/main/java/org/apache/accumulo/gc/GarbageCollectionEnvironment.java:
##########
@@ -72,6 +72,17 @@ public interface GarbageCollectionEnvironment {
*/
Stream<Reference> getReferences();
+ /**
+ * Return a list of all TableIDs in the
Review Comment:
Addressed in 1f8d37305789fa1540158e32b54da86a7af3ae87
--
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]
