Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Geoparquet support #153

Closed
haubourg opened this issue Oct 13, 2024 · 7 comments · Fixed by #174
Closed

Geoparquet support #153

haubourg opened this issue Oct 13, 2024 · 7 comments · Fixed by #174
Labels
feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed priority-medium Medium priority issue user-request This issue was directly requested by a user

Comments

@haubourg
Copy link

What feature are you requesting?

Hi, I'm testing pg_analytics for mostly geospatial use cases. Does pg_analytics supports geoparquet format as Crunchy bridge does ?
I saw only GeoJSON examples in the docs and failed to do geographic subset filters when trying to mimic this example : https://www.crunchydata.com/blog/vehicle-routing-with-postgis-and-overture-data

Why are you requesting this feature?

For a massive datawarehouse postgres with mostly geospatial data, we go toward analytics and explore all the available options to outsource storage of data to S3 geoparquet, and benefit from column storage features for analytics.
After having screened most available extensions, pg_analytics seems really advanced and easy to set up.
GeoParquet is now an Open Geospatial Consortium standard (OGC) and along with GeoPackage should be the first class citizens when dealing with spatial data.

What is your proposed implementation for this feature?

GDAL and duckDB both support GeoParquet reading and could provide the necessary bits.

Full Name:

Régis Haubourg

Affiliation:

CSTB

@haubourg haubourg added the feature New feature or request label Oct 13, 2024
@philippemnoel
Copy link
Collaborator

Hey @haubourg! Welcome :)

I don't believe we have Geoparquet support, but it should be easy to add by following our Parquet and GeoJSON supports if duckdb-rs supports it. Would you like to try your hand at it? We probably won't have time on our team to do this for quite some time.

@haubourg
Copy link
Author

Thanks for your feedback!
I unfortunately don't have the coding skills and my benevolent time goes to QGIS.org 😅.
Regards

@philippemnoel philippemnoel added good first issue Good for newcomers help wanted Extra attention is needed priority-medium Medium priority issue user-request This issue was directly requested by a user labels Oct 13, 2024
@DeoLeung
Copy link
Contributor

DeoLeung commented Nov 7, 2024

duckdb's spatial extension >=1.1.0 supports limited geoparquet, at least writing and reading geometry type is ok

@philippemnoel
Copy link
Collaborator

duckdb's spatial extension >=1.1.0 supports limited geoparquet, at least writing and reading geometry type is ok

Thanks for letting us know! Would you be willing to submit a patch for it? It should be really small

@DeoLeung
Copy link
Contributor

DeoLeung commented Nov 8, 2024

from python to rust requires some learning curve, just try cargo to bump it and ... learning why there are multiple versions in cargo.lock and ... the tests seems still installing v1.0.0 :(

@philippemnoel
Copy link
Collaborator

We are using duckdb-rs which is on duckdb 1.0.0. You' need to bump duckdb-rs to 1.1.0 before

@Youssef-Harby
Copy link

Hi @philippemnoel, is FDW for GeoParquet from S3 with geometry types supported now?
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed priority-medium Medium priority issue user-request This issue was directly requested by a user
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants