Skip to content

perf(rust/sedona-functions): implement ST_Boundary with geo-traits - #1256

Draft
paleolimbot wants to merge 1 commit into
apache:mainfrom
paleolimbot:codex/issue-303-faster-boundary
Draft

paleolimbot wants to merge 1 commit into
apache:mainfrom
paleolimbot:codex/issue-303-faster-boundary

Conversation

@paleolimbot

@paleolimbot paleolimbot commented Sep 10, 2026

Copy link
Copy Markdown
Member

Moves ST_Boundary from sedona-geos into the native sedona-functions set and replaces its GEOS round-trip with direct geo-traits traversal and incremental WKB output. Preserves GeometryCollection aggregation, dimensionality, empty outputs, and the mod-2 boundary node rule for MultiLineStrings. Moves the Criterion coverage into the native benchmark suite. Verified with all 564 sedona-functions tests, all 271 remaining sedona-geos tests, and clippy with warnings denied.

Closes #303

@github-actions
github-actions Bot requested a review from prantogg September 10, 2026 19:50
@paleolimbot
paleolimbot marked this pull request as draft September 11, 2026 03:39
@paleolimbot
paleolimbot force-pushed the codex/issue-303-faster-boundary branch from a366a01 to cdbfd3b Compare September 11, 2026 18:28
@paleolimbot paleolimbot changed the title perf(c/sedona-geos): implement ST_Boundary with geo-traits perf(rust/sedona-functions): implement ST_Boundary with geo-traits Sep 11, 2026
@paleolimbot
paleolimbot force-pushed the codex/issue-303-faster-boundary branch from cdbfd3b to 45790fe Compare September 12, 2026 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement faster ST_Boundary() using geo-traits

1 participant