stevie9868 commented on issue #1443: URL: https://github.com/apache/iceberg-rust/issues/1443#issuecomment-3127588225
@CTTY I have been thinking about how to do this. Currently there are two approaches I can think of: Option 1: Implement what iceberg java currently has by passing a term. We need to build `BoundTransform` and `UnBoundTransform`, and also the existing struct in term.rs naming is not quiet the same as iceberg java. For example `Reference` struct is called `NamedReference` in java. I was thinking to directly rename Reference to NamedReference and other naming as well to match the java iceberg naming but it might break the existing client. What is your recommendation here? Option 2: In our asc and desc function allow user to pass in Transform enum indicating the transform. Let me know what you think, thanks! -- 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]
