Opened 7 years ago

Closed 7 years ago

#3672 closed defect (fixed)

Change aggregate functions to use new syntax

Reported by: robe Owned by: robe
Priority: low Milestone: PostGIS 2.4.0
Component: postgis Version: master
Keywords: Cc:

Description

For this, it's not necessary to drop existing (so no need to put in a Changed: …). Internally PostgreSQL shows the new syntax anyway.

Just to keep our code base more up to date and make it easier to stuff args such as parallel = safe without having to rewrite the display.

Note that most of our postgis aggs look like old syntax:

CREATE AGGREGATE some_agg( basetype = geometry ... )

But the newer syntax has been supported since at least PostgreSQL 8.4

CREATE AGGREGATE some_agg(geometry)( ... )

https://www.postgresql.org/docs/8.4/static/sql-createaggregate.html

So we can certainly use in 2.1+

https://www.postgresql.org/docs/8.4/static/sql-createaggregate.html

Change History (2)

comment:1 by robe, 7 years ago

Owner: changed from pramsey to robe

comment:2 by robe, 7 years ago

Resolution: fixed
Status: newclosed

did this already as part of marking all as parallel safe.

Note: See TracTickets for help on using tickets.