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

Posted to github@arrow.apache.org by "Jefffrey (via GitHub)" <gi...@apache.org> on 2025/09/17 04:19:14 UTC

Re: [I] Improve the display of interval types [arrow-rs]

Jefffrey commented on issue #5914:
URL: https://github.com/apache/arrow-rs/issues/5914#issuecomment-3301251287

   Looks like there's an edge case where if all values are 0 for `MonthDayNano` then an empty string is used as the display:
   
   https://github.com/apache/arrow-rs/blob/2ec77b548d9242bc85ca36f04761d116ceab8963/arrow-cast/src/display.rs#L733-L758
   
   - Related DataFusion issue: https://github.com/apache/datafusion/issues/17455
   
   Could consider using `0 seconds` as the default, similar to what Spark does
   
   I think this issue might happen for `DayTime` too based on looking at the code:
   
   https://github.com/apache/arrow-rs/blob/2ec77b548d9242bc85ca36f04761d116ceab8963/arrow-cast/src/display.rs#L710-L731


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