You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/01 16:51:30 UTC

[GitHub] [arrow-datafusion] waitingkuo commented on issue #3997: change` date_part` return types to f64

waitingkuo commented on issue #3997:
URL: https://github.com/apache/arrow-datafusion/issues/3997#issuecomment-1298824543

   @comphead 
   
   `arrow-rs` uses `chrono-rs` to extract these date parts. Chono-rs does have [nanosecond](https://docs.rs/chrono/latest/chrono/trait.Timelike.html#tymethod.nanosecond) while arrow-rs doesn't have.
   
   We could extend `arrow-rs` to support `nanosecond` and then get back to this issue.
   
   I think adding nanosecond support in `arrow-rs` is a `good start issue`. If you're interested in, I'm happy to help you to figure out the codes to modify. Basically, you could follow the `second`'s logic to do so
   https://github.com/apache/arrow-rs/blob/master/arrow/src/compute/kernels/temporal.rs#L723-L764
   
   I could help to send a pr to arrow-rs as well


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org