alamb commented on issue #3100: URL: https://github.com/apache/arrow-datafusion/issues/3100#issuecomment-1212534680
Interval Proposal > 1. Consider make INTERVAL xxx outputs Interval(MonthDayNano) instead of Interval(DayTime) as it's easier to align with our Timestamp(NanoSecond, None) Here is the spec in case that is interesting: https://github.com/apache/arrow/blob/master/format/Schema.fbs#L354-L372 I think this is a good idea -- Interval(MonthDayNano) is a relatively new addition to the arrow standard, so I think DataFusion might default to `Interval(DayTime)` is because that was all that was available at the time of initial implementation. > 2. Carefully design the outputs for operators like what postgresql has Agree > 3. we could think about whether we really need timestamp with time zone - timestamp ... this is what postgresql has I don't quite understand the question about "really need timestamp with time zone - timestamp" -- as I thought your proposal for timezones was to fill out the feature set for timestamp with timezone 🤔 -- 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]
