Skip to content

Rename aggregate functions to have _Agg suffix #296

@paleolimbot

Description

@paleolimbot

As suggested by @jiayuasu:

We should rename this to ST_Polygonize_Agg. Sedona, Snowflake, Databricks all use _agg to separate their Spatial SQL aggregate functions from scalar functions.

https://sedona.apache.org/latest/api/sql/AggregateFunction/
https://docs.snowflake.com/en/sql-reference/functions-geospatial
https://docs.databricks.com/aws/en/sql/language-manual/sql-ref-st-geospatial-functions

This is also true for DuckDB.

ST_Polygonize_Agg has been updated; however I believe ST_Collect(), ST_Intersection_Aggr() and ST_Union_Aggr() still need to be updated (Jia also notes in that thread that Sedona will update its _Aggr suffix to be Agg to match everybody else as well.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions