moomindani commented on PR #2802: URL: https://github.com/apache/iceberg-rust/pull/2802#issuecomment-5111788263
Gentle ping — open for ~2.5 weeks, CI green, no review yet. This one is a correctness gap rather than a feature: the v3 spec serializes partition and sort fields whose transform takes multiple arguments with a `source-ids` list instead of a single `source-id`, and deserializing such metadata currently fails with `missing field source-id`. So a v3 table containing any multi-argument transform can't be read at all. Note it touches `public-api.txt`, so it's worth a look from that angle too. @blackmwk @CTTY would either of you have time for a review? Closes #2801, part of #2411. -- 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]
