zeroshade commented on PR #713: URL: https://github.com/apache/arrow-go/pull/713#issuecomment-4092197246
I agree with you that we should make them accessible to everyone. It looks like you've pretty much isolated everything into a single geoarrow package which is perfect and i think makes it ripe to be pulled out as its own package that can live in the geoarrow org (@paleolimbot could you help with creating a repo there for this?) where it can grow and extend to cover all of the geoarrow types and include packages for connecting to the wider geospatial ecosystem in go with geom-go etc. Then iceberg-go can simply import that geoarrow-go package and register the extension type to manipulate them. Given the scope of this I think that's probably the best route, particularly given the TODOs you have here for the remaining types. I wouldn't want to push this out here and then move it later on, it would be better to have that separate module from the beginning so it has room to grow. -- 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]
