anoopj opened a new pull request, #2843: URL: https://github.com/apache/iceberg-rust/pull/2843
## Which issue does this PR close? - Closes #2842. ## What changes are included in this PR? `ManifestsTable::scan` panicked when building the `partition_summaries` column for tables whose manifests reference a historical partition spec with a source column that has since been dropped from the current schema. This is a legal table state in v2+: once a partition field is removed from the default spec, its source column may be dropped. `PartitionSpec::partition_type` returns an error for such specs, and the `.unwrap()` turned it into a panic. The fix makes the failure a clean error rather than a crash. Making the manifests table render successfully for such tables (e.g. by resolving each spec against a compatible schema, as Java does) is left as a follow-up. ## Are these changes tested? Yes. A new unit test (`test_manifests_table_with_dropped_partition_source_column`) -- 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]
