paleolimbot opened a new pull request, #398:
URL: https://github.com/apache/sedona-db/pull/398
This doesn't quite work:
```
thread 'writer::test::geoparquet_1_1_with_sort_by_expr' panicked at
rust/sedona-geoparquet/src/writer.rs:817:14:
called `Result::unwrap()` on an `Err` value: Context("SanityCheckPlan",
Plan("Plan: [\"SortPreservingMergeExec: [sd_format(geometry@1) ASC NULLS
LAST]\", \" ProjectionExec: expr=[txt@0 as txt, geoparquet_bbox(geometry@1) as
bbox, geometry@1 as geometry]\", \" SortExec: expr=[sd_format(geometry@1)
ASC NULLS LAST], preserve_partitioning=[true]\", \" ProjectionExec:
expr=[CAST(sd_format(geometry@0) AS Utf8View) as txt, geometry@0 as
geometry]\", \" RepartitionExec: partitioning=RoundRobinBatch(12),
input_partitions=1\", \" DataSourceExec: file_groups={1 group:
[[Users/dewey/gh/sedona-db/submodules/geoarrow-data/ns-water/files/ns-water_water-point_geo.parquet]]},
projection=[geometry], file_type=parquet\"] does not satisfy order
requirements: [sd_format(geometry@1) ASC NULLS LAST]. Child-0 order:
[[sd_format(geometry@2) ASC NULLS LAST]]"))
```
I think the enforcement of ordering requirements is still moving things
around.
I'll try inlining the projection into some opaque datasink exec wrapper and
if that doesn't work or isn't easy we should just bump this to the next release.
--
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]
