Jefffrey commented on issue #4969: URL: https://github.com/apache/arrow-rs/issues/4969#issuecomment-1872906374
I don't think this is the correct way to go about implementing the `extract(x from interval)` functionality, since as mentioned there isn't a well defined way to convert an interval into a datetime. Would probably need to modify the existing temporal kernels: https://github.com/apache/arrow-rs/blob/c57857031fb39559924df56e1c8e5f60a1a6e69a/arrow-arith/src/temporal.rs#L225-L232 To be able to extract from intervals directly, instead of attempting to convert intervals to datetime to then leverage existing functionality of these kernels (which only currently work for datetimes). That or implement the logic entirely on Datafusion's side. Note there is an issue on Datafusion repo that relates to this functionality: https://github.com/apache/arrow-datafusion/issues/6327 -- 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]
