szehon-ho opened a new pull request, #3662: URL: https://github.com/apache/iceberg-rust/pull/3662
## Which issue does this PR close? Closes #3500. ## What changes are included in this PR? Allow fast append to use any partition spec retained by the table, so files written under spec 0 can still commit after the default changes to spec 1. Following Java FastAppend, write added files in separate manifests by spec and use each file's spec for validation and partition summaries. No public API changes. ## Are these changes tested? Unit tests cover mixed-spec appends in v1/v2/v3, unknown specs, incompatible partition values, duplicate paths, manifest reuse after the default spec changes, and multi-spec retries across a v1-to-v2 format upgrade. - `cargo test -p iceberg --lib`: 2,147 passed, 1 ignored. - `cargo clippy -p iceberg --all-targets --all-features -- -D warnings` - `cargo fmt --all -- --check` - `make generate-public-api`: no API changes. ## AI Disclosure Codex assisted with implementation and tests; Isaac (Claude) reviewed the change. Review focus: partition attribution in summaries and the assumption that cached manifests remain reusable when only the default spec changes. -- 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]
