milleruntime commented on code in PR #2767:
URL: https://github.com/apache/accumulo/pull/2767#discussion_r895747325
##########
core/src/main/java/org/apache/accumulo/core/gc/ReferenceDirectory.java:
##########
@@ -21,13 +21,22 @@
import org.apache.accumulo.core.data.TableId;
/**
- * Part of the Tablet File path that is definitely a directory.
+ * A GC reference to a Tablet directory, like t-0003.
*/
-public class ReferenceDirectory extends Reference {
- public final String tabletDir; // t-0003
+public class ReferenceDirectory extends ReferenceFile {
+ private final String tabletDir; // t-0003
public ReferenceDirectory(TableId tableId, String dirName) {
super(tableId, dirName);
Review Comment:
The metadataEntry will look like: `1;2cccccccccccccd1 srv:dir []
t-000003z`. So technically, the dirName is only part of the metadaEntry.
--
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: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
