efd6 opened a new issue, #445:
URL: https://github.com/apache/arrow-go/issues/445

   ### Describe the bug, including details regarding any error messages, 
version, and platform.
   
   
[`(*array.Timestamp).MarshalJSON`](https://github.com/apache/arrow-go/blob/513dc09a09384ae8ec8dad9ae4ea1ecc88680209/arrow/array/timestamp.go#L106-L113)
 calls 
[`(*array.Timestamp).ValueStr`](https://github.com/apache/arrow-go/blob/513dc09a09384ae8ec8dad9ae4ea1ecc88680209/arrow/array/timestamp.go#L90-L97)
 via 
[`(*array.Timestamp).GetOneForMarshal`](https://github.com/apache/arrow-go/blob/513dc09a09384ae8ec8dad9ae4ea1ecc88680209/arrow/array/timestamp.go#L99-L104)
 where the time format layout is "2006-01-02 15:04:05.999999999Z0700" which 
does not agree with the Go convention of using RFC3339 for JSON-encoded 
timestamp values (shown 
[here](https://cs.opensource.google/go/go/+/refs/tags/go1.24.5:src/time/format_rfc3339.go;l=62-80)
 called via 
[`time.Time.MarshalJSON`](https://cs.opensource.google/go/go/+/refs/tags/go1.24.5:src/time/time.go;l=1584-1593)).
   
   Doing this breaks downstream users of these JSON documents since most will 
use encoding/json, which expects valid RFC3339 timestamps 
([here](https://cs.opensource.google/go/go/+/refs/tags/go1.24.5:src/time/format_rfc3339.go;l=155-188)
 via 
[here](https://cs.opensource.google/go/go/+/refs/tags/go1.24.5:src/time/time.go;l=1595-1609)).
   
   ### Component(s)
   
   Parquet


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