Re: Function definition regression in 15beta1 when specific parameter name (string) is used - Mailing list pgsql-general

From Alastair McKinley
Subject Re: Function definition regression in 15beta1 when specific parameter name (string) is used
Date
Msg-id PAXPR02MB760077B509448C42D2CB7093E3DA9@PAXPR02MB7600.eurprd02.prod.outlook.com
Whole thread Raw
In response to Re: Function definition regression in 15beta1 when specific parameter name (string) is used  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-general
> From: Tom Lane <tgl@sss.pgh.pa.us>
> Sent: 29 May 2022 18:43
> To: Alastair McKinley <a.mckinley@analyticsengines.com>
> Cc: Andrew Dunstan <andrew@dunslane.net>; pgsql-general@lists.postgresql.org <pgsql-general@lists.postgresql.org>
> Subject: Re: Function definition regression in 15beta1 when specific parameter name (string) is used
>
> Alastair McKinley <a.mckinley@analyticsengines.com> writes:
> > The following function definition fails in 15beta1 (ok in 14.3):
>
> >     create or replace function regexp_match_test(string text,pattern text) returns text[] as
> >     $$
> >         select regexp_match(string,pattern);
> >     $$ language sql;
>
> Commit 1a36bc9db seems to have defined STRING as a type_func_name_keyword,
> which strikes me as a pretty horrible trampling on user namespace.  That
> means you can't have tables or columns named "string" anymore either, and
> I'll bet money the latter restriction is going to bite a lot of people.
>

Yes I would agree, could this potentially break a lot of upgrades?

I checked the release notes and CTRL-F'd for "string" to check in case it had become reserved or become an alias for
text,but there is nothing in the release notes at the minute. 

> In a quick experiment here, I don't see any bison complaints if I
> back it down to unreserved_keyword, so this seems easily fixable.
> I wonder though if we don't need more review of patches that add
> partially- or fully-reserved keywords.
>
>                         regards, tom lane


pgsql-general by date:

Previous
From: Tom Lane
Date:
Subject: Re: Function definition regression in 15beta1 when specific parameter name (string) is used
Next
From: Alastair McKinley
Date:
Subject: psql 15beta1 does not print notices on the console until transaction completes