felixYyu commented on a change in pull request #4438:
URL: https://github.com/apache/iceberg/pull/4438#discussion_r840153023
##########
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 {@link ManifestFile} files to explore for this file's
metadata table scan
+ * @return list of manifest files
Review comment:
the static check warning log ，so I think should need change:
```
[MissingSummary] A summary fragment is required; consider using the value of
the @return block as a summary fragment instead.
* @return list of manifest files to explore for this files metadata
table scan
^
(see
https://google.github.io/styleguide/javaguide.html#s7.2-summary-fragment)
Did you mean '*Returns list of manifest files to explore for this files
metadata table scan.'?
```
--
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]
