fallintoplace opened a new pull request, #1025:
URL: https://github.com/apache/arrow-go/pull/1025

   ### Rationale for this change
   
   Timestamp arrays can declare a timezone, but the CSV writer formats every 
value directly in UTC. This produces the wrong local timestamp and also lets 
invalid timezone declarations pass silently.
   
   ### What changes are included in this PR?
   
   * Use the timestamp type conversion function so values are formatted in the 
declared timezone.
   * Return invalid timezone errors from Writer.Write.
   * Propagate conversion errors from nested list values.
   
   ### Are these changes tested?
   
   Yes. The tests verify epoch formatting in America/New_York and rejection of 
an invalid timezone. All CSV tests and the assertion build pass with the 
required Parquet test data.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to