ctubbsii commented on code in PR #3476:
URL: https://github.com/apache/accumulo/pull/3476#discussion_r1224538767
##########
core/src/main/java/org/apache/accumulo/core/metadata/TabletFile.java:
##########
@@ -20,15 +20,26 @@
import org.apache.hadoop.fs.Path;
+/**
+ * An interface that represents different types of file references that are
handled by code that
+ * processes tablet files.
+ */
public interface TabletFile {
/**
- * @return The file name of the TabletFile
+ * Returns the fileName of the TabletFile. The value return is the name
itself and not the entire
+ * path.
+ *
+ * For example, if the full path for a TabletFile is
+ * 'hdfs://nn1/accumulo/tables/5a/t-0001/F0002.rf', this method returns
'F0002.rf'.
*/
String getFileName();
/**
- * @return The path of the TabletFile
+ * Returns the full path for the TabletFile on the file system. The path may
be normalized
+ * depending on the specific implementation. For example, a path in hdfs
would be returned as
+ * `hdfs://nn1/accumulo/tables/5a/t-0001/F0002.rf`
Review Comment:
```suggestion
* 'hdfs://nn1/accumulo/tables/5a/t-0001/F0002.rf'
```
--
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]
