alamb commented on PR #5140:
URL:
https://github.com/apache/arrow-datafusion/pull/5140#issuecomment-1412143332
> The downside for now is additional cast created sometimes for timestamp,
to convert Timestamp(_, Some("00:00")) to Timestamp(_, None). This can be
solved in separate PR if you dont mind as it might again require changes in
arrow-rs
I think this is correct (and needed) if the timestamps in question have
different timezones. It should not really effect runtime performance as I
expect both to be evaluated at plan time
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
