SaymV commented on PR #2859: URL: https://github.com/apache/iceberg-python/pull/2859#issuecomment-3761944547
Thank you for the reviews! This PR is the groundwork. The next phase, building on the existing GeometryType/GeographyType foundation, will be: 1. **Integration Tests & Round-Trip** - End-to-end verification 2. **Bounds Metrics** - WKB parsing for [query optimization](https://iceberg.apache.org/spec/#bounds-for-variant-geometry-and-geography). 3. **Spatial Predicate Helpers** - Expressions to use the stored bounds metrics (ST_Contains, ST_Intersects, ST_Within, ST_Overlaps) I'll be sure to gate the dependencies with the modularity `[geospatial]`. My interest in this functionality is mostly personal, I've been playing around with exported Google Timeline data (that they provide in json via [Google Takeout](https://takeout.google.com/) and would like to set something up for regular export and storage in a much more performant and stable format. Happy to change/move/remove the documentation I supplied also. I felt like the RFC was necessary to include for discussion, but I'm not sure about the team's desire to keep those docs in the repo. -- 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]
