bjchambers edited a comment 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 down" to the number of days). Or perhaps both (cast to 64-bit value preserves both parts, cast to 32-bit value returns the days)? Another option would be to convert to a `f64`, where the integer part is the number of days, and the fractional part is the ratio of the millisecond part to the number of milliseconds in a day. -- 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]
