u70b3 opened a new pull request, #2723: URL: https://github.com/apache/iceberg-rust/pull/2723
## Which issue does this PR close? - Closes #2722. ## What changes are included in this PR? - Add writer config validation for equality delete field IDs. - Reject empty equality ID lists, duplicate equality field IDs, and field IDs that are missing from the table schema. - Keep the existing `RecordBatchProjector` behavior for unsupported field types and map/list reachability. - Fix a stale comment: equality delete fields may be optional, unlike identifier fields. ## Are these changes tested? - `cargo test -p iceberg test_equality_delete_rejects_empty_equality_ids --lib` - `cargo test -p iceberg test_equality_delete_rejects_duplicate_equality_ids --lib` - `cargo test -p iceberg test_equality_delete_rejects_missing_equality_id --lib` - `cargo test -p iceberg test_equality_delete_unreachable_column --lib` - `cargo test -p iceberg test_equality_delete_with_nullable_field --lib` - `cargo test -p iceberg equality_delete --lib` - `cargo fmt --check` - `cargo test -p iceberg --lib` - `cargo clippy -p iceberg --all-features --lib --tests -- -D warnings` - `make check` - `make unit-test` Also attempted `make test`, but the local run did not reach tests because Docker image pull from Docker Hub timed out while starting `dev/docker-compose.yaml`. -- 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]
