jagdeeps91 commented on issue #718: URL: https://github.com/apache/iceberg-rust/issues/718#issuecomment-2846052113
I have a pull request for this issue: https://github.com/apache/iceberg-rust/pull/1285 This just removes this if/else check: https://github.com/apache/iceberg-rust/blob/main/crates/iceberg/src/spec/partition.rs#L582 As per spec the source column should be primitive type for all Transforms. @c-thiel As original author, please let me know if this check should not be removed. This is my first PR to iceberg-rust, would be very happy to get feedback! -- 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]
