ajaypadwal73 commented on PR #990: URL: https://github.com/apache/sedona-db/pull/990#issuecomment-4794567494
@paleolimbot Thanks - addressed the latest round: **ST_DelaunayTriangles** - Switched the flags matcher from `is_boolean` to `is_integer`. - Added a Rust test covering the flags (MULTILINESTRING output) branch. - Docs: updated the description to mention the `flags` parameter and added a flags example. **ST_ExteriorRing** - Reworked `invoke_scalar` to write GEOS output directly into the builder via `mut impl Write`, avoiding the intermediate WKB buffer copy. - Fixed the test assertion and docs (description + metadata) to use LINESTRING instead of LINEARRING. **ST_PointOnSurface** - Dropped the geography matcher — it's a planar algorithm and geography is already implemented separately. **ST_BuildArea** - Now catches the non-linework (POINT) case and returns NULL, matching PostGIS. Let me know if anything else needs tweaking. -- 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]
