prochac commented on issue #813:
URL: https://github.com/apache/arrow-go/issues/813#issuecomment-4440805238

   @zeroshade I did a dry-run. (Although I need to double-check.)
   The only braking change is 
   
https://pkg.go.dev/github.com/apache/arrow/go/v17/arrow/avro#ArrowSchemaFromAvro
   Because its argument points directly to 
   https://pkg.go.dev/github.com/hamba/avro/v2#Schema
   
   We have two options, both adds 
   `func ArrowSchemaFromAvroJSON(string) (*arrow.Schema, error)` 
   People using `hamba/avro` can use 
`ArrowSchemaFromAvroJSON(hambaAvroSchema.String())` instead of 
`ArrowSchemaFromAvro(hambaAvroSchema)`
   
   A: remove `ArrowSchemaFromAvro` in a favor of `ArrowSchemaFromAvroJSON`
   B: keep `ArrowSchemaFromAvro` for performance reasons, but treat it as 
unstable for the future (if another lib change comes)


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