jorisvandenbossche commented on pull request #10176:
URL: https://github.com/apache/arrow/pull/10176#issuecomment-836934637


   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)


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


Reply via email to