paleolimbot opened a new pull request, #8520: URL: https://github.com/apache/arrow-rs/pull/8520
# Which issue does this PR close? - Closes #7799. (The other issue, #7240 I think would cover the Arrow half that Kyle has prototyped in #8222). This PR is a subset of #8222 (just the part where the `crs` and `algorithm` fields were added) that also includes tests. # Rationale for this change When the Geography and Geometry members of the LogicalType enum were added, they did not include the underlying parameters. Without the parameters, low-level clients to the Parquet library like SedonaDB can't read Parquet files that contain these types without loosing information. # What changes are included in this PR? This PR adds the `crs` and `algorithm` parameters to the`Geometry` and `Geography` logical types and updates `match` statements with reasonable implementations that roundtrip these values. # Are these changes tested? Yes # Are there any user-facing changes? Yes: any match statements written against the `LogicalType` will break. This is a breaking change. -- 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]
