pvary commented on code in PR #11513:
URL: https://github.com/apache/iceberg/pull/11513#discussion_r1941147406
##########
core/src/main/java/org/apache/iceberg/actions/RewriteFileGroup.java:
##########
@@ -31,26 +31,26 @@
import org.apache.iceberg.util.DataFileSet;
/**
- * Container class representing a set of files to be rewritten by a
RewriteAction and the new files
- * which have been written by the action.
+ * Container class representing a set of data files to be rewritten by a
RewriteAction and the new
+ * files which have been written by the action.
*/
-public class RewriteFileGroup {
- private final FileGroupInfo info;
- private final List<FileScanTask> fileScanTasks;
-
+public class RewriteFileGroup extends FileRewriteGroup<FileGroupInfo,
FileScanTask, DataFile> {
Review Comment:
Maybe `RewriteDataFileGroup` could be better?
--
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]
