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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/07/02 00:48:13 UTC

[GitHub] [arrow] rok commented on pull request #10610: ARROW-13033: [C++] Kernel to localize naive timestamps to a timezone (preserving clock-time)

rok commented on pull request #10610:
URL: https://github.com/apache/arrow/pull/10610#issuecomment-872636186


   Thanks for the review @jorisvandenbossche :).
   I've gone through your feedback and done some of the boilerplate except for the correct output timestamp type. I'll try to do it over the weekend.
   
   > (also, given all the confusion also on the JIRA, we should probably try to come up with a more explicit/descriptive name ...)
   
   I think there is no consensus on this yet. `tz_convert`, `tz_localize` were mentioned. I'd add `tz_normalize`. Any other?


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