[Geospatial]: Scaffolding for new parquet-geospatial crate#8375
[Geospatial]: Scaffolding for new parquet-geospatial crate#8375alamb merged 3 commits intoapache:mainfrom
parquet-geospatial crate#8375Conversation
parquet-geometry crate
paleolimbot
left a comment
There was a problem hiding this comment.
Awesome!
Feel free to lift SedonaDB's Interval and BoundingBox implementation, which is the only one I know about that implements the "wraparound" version of the bounding box we adopted in the spec.
https://github.com/apache/sedona-db/blob/main/rust/sedona-geometry/src/interval.rs
https://github.com/apache/sedona-db/blob/main/rust/sedona-geometry/src/bounding_box.rs
...or if there's a more appropriate place to upstream those I can do that too. (I wrote that code and am happy to do the PR if that's cleaner IP wise).
alamb
left a comment
There was a problem hiding this comment.
Makes sense to me -- thank you @kylebarron . I double checked most of the links and they are 👌
A good follow on PR might be to add CI checks for this crate, similar to https://github.com/apache/arrow-rs/blob/main/.github/workflows/parquet-variant.yml
|
how about naming it |
|
Or |
|
We used a |
parquet-geometry crateparquet-geospatial crate
parquet-geospatial crateparquet-geospatial crate
|
Thanks everyone. Onwards to geospatial support! |
Which issue does this PR close?
parquet-geometrycrate #8374.Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?