Skip to content

feat: implement RewriteFiles operation#751

Merged
wgtmac merged 7 commits into
apache:mainfrom
WZhuo:rewrite_files
Jun 30, 2026
Merged

feat: implement RewriteFiles operation#751
wgtmac merged 7 commits into
apache:mainfrom
WZhuo:rewrite_files

Conversation

@WZhuo

@WZhuo WZhuo commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

This PR implements the RewriteFiles operation for the Iceberg C++ library.

Changes

  • Added RewriteFiles update class in src/iceberg/update/rewrite_files.h and .cc
  • Added RewriteFiles to the type forward declarations in src/iceberg/type_fwd.h
  • Registered RewiteFiles as a merging snapshot update in src/iceberg/table.h and .cc
  • Added RewriteFiles support in transaction (src/iceberg/transaction.h and .cc)
  • Added comprehensive tests in src/iceberg/test/rewrite_files_test.cc
  • Added merge test for RewriteFiles in src/iceberg/test/merging_snapshot_update_test.cc

@WZhuo WZhuo force-pushed the rewrite_files branch 4 times, most recently from d28ffe9 to b24cd4d Compare June 16, 2026 12:18
Comment thread src/iceberg/table.cc
Comment thread src/iceberg/update/rewrite_files.cc Outdated
Comment thread src/iceberg/test/rewrite_files_test.cc Outdated
Comment thread src/iceberg/update/rewrite_files.h
Comment thread src/iceberg/test/rewrite_files_test.cc
Comment thread src/iceberg/test/rewrite_files_test.cc Outdated
@WZhuo WZhuo force-pushed the rewrite_files branch 3 times, most recently from 4f87a5c to 603c1da Compare June 29, 2026 09:49
@wgtmac

wgtmac commented Jun 30, 2026

Copy link
Copy Markdown
Member

Thanks @WZhuo for adding this and @zhjwpku for the review!

@wgtmac wgtmac merged commit 61400e8 into apache:main Jun 30, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants