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