C-Loftus commented on PR #960: URL: https://github.com/apache/arrow-go/pull/960#issuecomment-5024593377
Thanks @dwilson1988 for your comment! I have been discussing with @zeroshade in the iceberg-go slack if it would be relevant https://apache-iceberg.slack.com/archives/C05J3MJ42BD/p1784555139111839 Essentially I think my plan will be to get this merged, then in geoarrow-go we can implement `ParquetLogicalType` so that the geoarrow structs get picked up as the GeometryType i am currently working on. That way the geoarrow logic is kept out of this repo but should still work in the write path. For read that will be a follow up issue from this. According to @zeroshade > essentially you'd probably need to add something in https://github.com/apache/arrow-go/blob/main/arrow/datatype_extension.go#L43 to allow iterating the registered extension types. then add to https://github.com/apache/arrow-go/blob/main/parquet/pqarrow/schema.go#L528 cases for geography and geometry, where you iterate the registered extension types and use one that matches the logical type you got, which will need to add a new interface to look for to map from the logical type to the extension type -- 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]
