bjchambers commented on issue #685:
URL: https://github.com/apache/arrow-rs/issues/685#issuecomment-896946545


   For the most part this seems straightforward to add -- the only weird one is 
`IntervalDayTime`. The underlying value is an i64 representing the 32-bit days, 
and the 32-bit milliseconds. A naive cast would produce a somewhat meaningless 
64-bit integer value. An alternative would be to allow casting it to the 32-bit 
days (in some sense "rounding it" to the number of days). Or perhaps both (cast 
to 64-bit value preserves both parts, cast to 32-bit value returns the days)?


-- 
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]


Reply via email to