Re: [PR] feat: Add Geometry & Geography Types [iceberg-python]

["SaymV (via GitHub)" <gi...@apache.org>]

SaymV commented on code in PR #2859:
URL: https://github.com/apache/iceberg-python/pull/2859#discussion_r2760367438


##########
mkdocs/docs/geospatial.md:
##########
@@ -0,0 +1,110 @@
+# Geospatial Types
+
+PyIceberg supports Iceberg v3 geospatial primitive types: `geometry` and `geography`.
+
+## Overview
+
+Iceberg v3 introduces native support for spatial data types:
+
+- **`geometry(C)`**: Represents geometric shapes in a coordinate reference system (CRS)
+- **`geography(C, A)`**: Represents geographic shapes with CRS and calculation algorithm
+
+Both types store values as WKB (Well-Known Binary) bytes.
+
+## Requirements
+
+- Iceberg format version 3 or higher
+- `geoarrow-pyarrow` for full GeoArrow extension type support (optional: `pip install pyiceberg[geoarrow]`)

Review Comment:
   Good catch, it’s optional. I clarified the Requirements section to note that `geoarrow-pyarrow` is only needed for GeoArrow extension type metadata/interoperability; without it, geometry/geography are written as binary in Parquet while the Iceberg schema still preserves the spatial 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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org