-
Notifications
You must be signed in to change notification settings - Fork 45
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
Inconsistent behavior of ST_READ in importing EMPTY geometry #273
Labels
Comments
rouault
added a commit
to rouault/gdal
that referenced
this issue
Mar 9, 2024
…epresentation of POLYGON EMPTY (fixes duckdb/duckdb-spatial#273) This was what we exported already, which also happens to validate against http://geojson.org/schema/Polygon.json but on import, we insisted on havin a [[]] coordinates array (which does not validate the schema)
assuming DukDB Spatial uses GDAL for GeoJSON parsing, this will be fixed per OSGeo/gdal#9437 |
rouault
added a commit
to rouault/gdal
that referenced
this issue
Mar 9, 2024
…epresentation of POLYGON EMPTY (fixes duckdb/duckdb-spatial#273) This was what we exported already, which also happens to validate against http://geojson.org/schema/Polygon.json but on import, we insisted on havin a [[]] coordinates array (which does not validate the schema)
rouault
added a commit
to OSGeo/gdal
that referenced
this issue
Mar 15, 2024
GeoJSON reader: accept a {"type": "Polygon","coordinates": []} as a representation of POLYGON EMPTY (fixes duckdb/duckdb-spatial#273)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Considering the following geojson file:
and SQL statement:
The EMPTY POLYGON should be imported into t0 to avoid losing information during the importing process.
Besides, the EMPTY LINESTRING is imported successfully. The importing behaviors of POLYGON and LINESTRING should be consistent.
Version:
Spatial version:
FORCE INSTALL spatial FROM 'http://nightly-extensions.duckdb.org'; and LOAD spatial;
DuckDB version:
The text was updated successfully, but these errors were encountered: