Commit d9bb013
authored
Adds ExtensionType for Parquet geospatial WKB arrays (#8943)
# Which issue does this PR close?
This does not fully close, but is an initial component of:
- #8717
# Rationale for this change
To keep PR size digestible, this implements the bi-directional extension
type metadata parsing for Parquet geospatial arrays. An actual array
type that allows users to easily interact with the data can come as a
follow-on PR.
# What changes are included in this PR?
- Implements the ExtensionType for Parquet geospatial logical types
- Adds GeoArrow compatible Metadata type to hold geospatial metadata
- Adds basic tests around geospatial metadata
serialization/deserialization
- Integrates geospatial logical type parsing into schema extensions
# Are these changes tested?
Yes. Targeted unit tests for the metadata and associated parsing have
been implemented. Higher level tests that show a full round-trip to and
from arrays can be included in the array implementation PR.
# Are there any user-facing changes?
Yes. All new public items have been documented, and no API breaking
changes have been made.
##
cc @alamb @paleolimbot @kylebarron1 parent 63212d2 commit d9bb013
File tree
7 files changed
+689
-131
lines changed- parquet-geospatial
- src
- parquet
- src
- arrow/schema
- tests
7 files changed
+689
-131
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
| 34 | + | |
| 35 | + | |
34 | 36 | | |
35 | 37 | | |
36 | 38 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
0 commit comments