omerhadari commented on issue #933: URL: https://github.com/apache/iceberg-rust/issues/933#issuecomment-2639357983
@liurenjie1024 this is how I worked around it locally :) Didn't want to get into implementation before getting an opinion because it is admittedly workaround-y in nature, and also is only good for a subset of use cases (stuff like interval math, or other datetime/date/time types are not easily handled by this). However I think using `TO_DATE` and `CAST(... AS DATE)` are common enough use cases to warrant a fix. I think the concern I mentioned with `CAST` expressions is a bit more worrying though, as it could cause incorrect results - not just slower queries. What do you think? I don't know if there are more examples of casts that also transform the value. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
