moomindani commented on PR #2790: URL: https://github.com/apache/iceberg-rust/pull/2790#issuecomment-5111788044
Gentle ping — this has been open for ~3 weeks with CI green and no review yet. It's a small, self-contained change: `FromStr for Transform` currently errors on any unrecognized transform name, but the v3 spec requires readers to read tables with unknown transforms and ignore them, so a table using a transform this version doesn't know about is unreadable today. One file, `spec/transform.rs`. Scoped deliberately to the non-breaking subset — preserving the original name for round-trip needs a `Transform::Unknown(String)` API discussion, which stays in #2789. @blackmwk you've reviewed most of the recent `spec` changes — would you mind taking a look when you have a moment? 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]
