RussellSpitzer commented on code in PR #11555:
URL: https://github.com/apache/iceberg/pull/11555#discussion_r1851031865


##########
api/src/main/java/org/apache/iceberg/actions/RewriteTablePath.java:
##########
@@ -91,9 +92,10 @@ interface Result {
     String stagingLocation();
 
     /**
-     * Path to a comma-separated list of source and target paths for all files 
added to the table
-     * between startVersion and endVersion, including original data files and 
metadata files
-     * rewritten to staging.
+     * Result file list location. This file contains a 'copy plan', a 
comma-separated list of all

Review Comment:
   This still feels a little ambiguous. 
   
   Maybe?
   
   A file containing a listing of both original file names and file names under 
the new prefix, comma separated. 



-- 
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

Reply via email to