Re: Removing WITH clause support in CREATE FUNCTION, for isCachable and isStrict
Michael Paquier <michael.paquier@gmail.com>
From: Michael Paquier <michael.paquier@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Daniel Gustafsson <daniel@yesql.se>, Postgres hackers <pgsql-hackers@postgresql.org>
Date: 2018-01-26T01:12:54Z
Lists: pgsql-hackers
On Thu, Jan 25, 2018 at 06:30:04PM -0500, Tom Lane wrote: > I poked into the git log and confirmed Michael's statement that > CREATE FUNCTION ... WITH has been documented as deprecated since > 7.3 (commit 94bdc4855 to be exact). Thanks for the confirmation. > I think the original intention was that we'd keep it for PG-specific > function attributes (ie those not found in the SQL spec), so as to > avoid creating keywords for every attribute we thought of. That > intention has been roundly ignored since then, though, since COST > and ROWS and LEAKPROOF and PARALLEL (UN)SAFE have all been implemented > as separate keywords not WITH options. I'm dubious that that was a good > plan, but it's probably too late to change direction now. :( > In short, I'm on board with removing the WITH clause. I've not > reviewed the patch in detail, but will do so and push it if there's > not objections pretty soon. Glad to hear that, thanks! -- Michael
Commits
-
Remove the obsolete WITH clause of CREATE FUNCTION.
- 4971d2a32209 11.0 landed
-
Extend syntax of CREATE FUNCTION to resemble SQL99.
- 94bdc4855cf8 7.3.1 cited