RussellSpitzer commented on a change in pull request #4376:
URL: https://github.com/apache/iceberg/pull/4376#discussion_r837632937
##########
File path: core/src/main/java/org/apache/iceberg/BaseFilesTable.java
##########
@@ -108,7 +108,8 @@ public TableScan appendsAfter(long fromSnapshotId) {
}
/**
- * @return list of manifest files to explore for this files metadata table
scan
+ * Returns list of manifest files to explore for this file's metadata
table scan
Review comment:
For this kind of one line doc, we just do something like
```
/**
* Returns the {@link Schema} for this sort order
*/
```
If you don't flag Return as `@return` then you'll make the Javadoc with
a summary and the function's return type will be clear
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
