Jefffrey commented on PR #23279: URL: https://github.com/apache/datafusion/pull/23279#issuecomment-4993288918
my main concern now is that we always cast the time to `time(ns)` when adding an interval. i was hoping we could try preserve the time unit, but thinking on it more we run into edge cases like when if `time(s) + interval '1 nanosecond'` 🤔 im not sure looking to other systems can help us here since i dont think they have different granularities for their time types like we do here -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
