geoarrow-rs icon indicating copy to clipboard operation
geoarrow-rs copied to clipboard

Published 20 hours ago • verified publisher icon geoarrow

Reexport geos/geozero/gdal/proj crates

Open lewiszlw opened this issue 1 year ago • 2 comments

lewiszlw avatar Dec 15 '23 06:12 lewiszlw

Can you give some context about why we should do this? Presumably this is to simplify version resolving when a third party user is also using these crates, but a user will still have to be alert whenever we bump one of these versions. If a user wants whatever version is compatible with geoarrow-rs, why not set gdal = "*" and let cargo resolve based on this lib?

We use lots of dependencies... why stop at these four? Are we going to re-export every arrow dependency as well? (to be clear, this is devil's advocate; I'm inclined to not re-export any crate)

kylebarron avatar Dec 15 '23 14:12 kylebarron

I think it's convenient for users that reexporting same ecosystem crates. For example, datafusion reexports arrow/parquet.

lewiszlw avatar Dec 16 '23 10:12 lewiszlw

I'd prefer to not re-export for now. It's easier to add this in the future than to take it away.

kylebarron avatar Aug 26 '24 19:08 kylebarron