-
Notifications
You must be signed in to change notification settings - Fork 689
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support as_datetime::<DataType::Interval~> #4969
Comments
A DateTime is a point in time, whereas an interval is not, I'm not entirely sure how this would work? |
I don't think this is the correct way to go about implementing the Would probably need to modify the existing temporal kernels: arrow-rs/arrow-arith/src/temporal.rs Lines 225 to 232 in c578570
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: apache/datafusion#6327 |
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I'd like to call
as_datetime::<DataType::Interval~>
to address apache/datafusion#7097.Is it possible to return
Some(~)
, notNone
?:arrow-rs/arrow-array/src/temporal_conversions.rs
Line 216 in 03d0505
Also, is it possible to uncomment the following lines?:
arrow-rs/arrow-array/src/types.rs
Lines 299 to 301 in 03d0505
Describe the solution you'd like
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: