emkornfield edited a comment on pull request #7604: URL: https://github.com/apache/arrow/pull/7604#issuecomment-653081545
> > > For timezone naive Timestamps, I'm not sure if we should be adjusting the datetime to reflect UTC converted to system time zone. thoughts? > > > > > > I don't think we should do that (it would also be a big break). Tz-naive timestamps have no timezone and should therefore be left as is IMO (applications on top of Arrow can choose to use the "naive means system time zone" behaviour, but I would say that Arrow itself should never deal with the system time zone) > > You've probably seen this, but this is what our docs say about timezone-naive data: http://arrow.apache.org/docs/cpp/api/datatype.html#_CPPv4N5arrow13TimestampTypeE > > (Side note: do our doc URLs have to be so ugly?) > > I agree with Joris that at this level, we should not do any localizing of timezone-naive data. Please see reply: https://github.com/apache/arrow/pull/7604#discussion_r449096333 I haven't looked in the code deeply but this might an overall bug, or at least idiosyncrasy, with our conversion to/from datetime (not accounting for system timezone). We can handle this as a separate issue though. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
