pitrou commented on code in PR #37788:
URL: https://github.com/apache/arrow/pull/37788#discussion_r1334789755
##########
go/arrow/cdata/cdata.go:
##########
@@ -197,7 +197,7 @@ func importSchema(schema *CArrowSchema) (ret arrow.Field,
err error) {
// handle types with params via colon
typs := strings.Split(f, ":")
- defaulttz := "UTC"
+ defaulttz := ""
Review Comment:
Yes, it was incorrect and failing the integration tests between C++ and Go.
A timestamp with an missing timezone is not the same as a timestamp with the
UTC timezone, as explained in the format spec:
https://github.com/apache/arrow/blob/772a01c080ad57eb11e9323f5347472b769d45de/format/Schema.fbs#L264
--
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]
