drbothen commented on issue #1607: URL: https://github.com/apache/iceberg-rust/issues/1607#issuecomment-3904976649
Hi @CTTY, I've been working on a RewriteFiles implementation for our compaction use case and have a working version I'd like to contribute. Before opening a PR, I wanted to check with you since you own this issue: 1. Are you actively working on #1606? 2. Would you be open to reviewing a PR that implements the core `RewriteFilesAction` + `SnapshotProducer` delete file support? The implementation covers: - `SnapshotProducer` changes to support `deleted_data_files` (write deleted manifest, validate files exist, summary tracking) - `RewriteFilesAction` that uses `Operation::Replace` per the spec - Validation that files to delete exist in the current snapshot - 6 unit tests including round-trip with catalog Happy to coordinate however works best! -- 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]
