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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to