dramaticlly commented on code in PR #12844:
URL: https://github.com/apache/iceberg/pull/12844#discussion_r2052716230
##########
api/src/main/java/org/apache/iceberg/actions/RewriteTablePath.java:
##########
@@ -86,6 +86,18 @@ public interface RewriteTablePath extends
Action<RewriteTablePath, RewriteTableP
*/
RewriteTablePath stagingLocation(String stagingLocation);
+ /**
+ * Allows the user to skip saving the file list, determining whether certain
files should be
+ * skipped from being saved.
+ *
+ * @param skipFileList A boolean value indicating whether to skip file
saving. Pass true to skip,
+ * false to not skip.
+ * @return this for method chaining
+ */
Review Comment:
I feel we can be more concise about it since this is a boolean flag, how
about?
```
/**
* Whether to skip saving the file list location.
*
* @param skipFileList true to skip saving the file list, false to include
it
* @return this instance for method chaining
*/
```
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
