rok commented on pull request #10176: URL: https://github.com/apache/arrow/pull/10176#issuecomment-840118305
> General comment: the PR is quite big, and the timezone-related logic further added complexity. So from a workflow perspective (to ensure reviewing / getting this merged is manageable), could it make sense to keep the timezone-related logic for a separate PR? (didn't check the code to see whether that's actually feasible) Keeping tz-aware logic would be fairly easy to move to a separate PR. I'll move it out. -- 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: [email protected]
