joonaspessi opened a new pull request, #286:
URL: https://github.com/apache/sedona-db/pull/286
Implements `ST_Polygonize` aggregate function following PostGIS behavior.
Creates polygons from a collection of linestrings that form closed rings using
the GEOS polygonize algorithm.
This is the first GEOS-based aggregate function in the codebase and asking
for opinions about the implementation as I'm quite unsure if this is the
correct approach.
## How to support `geometry[]` array input?
PostGIS supports two function signatures:
```sql
-- Aggregate form (implemented)
SELECT ST_Polygonize(geom) FROM table;
-- Array form (NOT implemented)
SELECT ST_Polygonize(ARRAY[geom1, geom2, geom3]);
```
Any ideas how to implement this with DafaFusion and keep support for both
input styles?
Should we separate the "array form" to another PR?
--
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]
