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

From Tom Lane
Subject Re: Function definition regression in 15beta1 when specific parameter name (string) is used
Date
Msg-id 3498459.1653846197@sss.pgh.pa.us
Whole thread Raw
In response to Function definition regression in 15beta1 when specific parameter name (string) is used  (Alastair McKinley <a.mckinley@analyticsengines.com>)
Responses Re: Function definition regression in 15beta1 when specific parameter name (string) is used
List pgsql-general
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.

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: Adrian Klaver
Date:
Subject: Re: Function definition regression in 15beta1 when specific parameter name (string) is used
Next
From: Alastair McKinley
Date:
Subject: Re: Function definition regression in 15beta1 when specific parameter name (string) is used