Re: function defined (or not), more worries on version 10->14 upgrade

From: Rob Sargent <robjsargent(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-general(at)postgresql(dot)org
Subject: Re: function defined (or not), more worries on version 10->14 upgrade
Date: 2022-04-15 20:01:36
Message-ID: 024A7C75-C1D5-41CF-B68F-23F418E8379A@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general

> On Apr 15, 2022, at 11:14 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Rob Sargent <robjsargent(at)gmail(dot)com> writes:
>> The function in question is working but I'm having trouble getting at
>> it's current definition and the possibility that there are more v10->v14
>> issues in this database. The last ERROR: below seems to me "a system
>> thing"(tm)
>
> Hmm, it looks like \df does not cope well with spaces in the argument
> list:
>
> regression=# \df genome_threshold_mono(text,text,double)
> List of functions
> Schema | Name | Result data type | Argument data types | Type
> --------+------+------------------+---------------------+------
> (0 rows)
>
> regression=# \df genome_threshold_mono(text,text, double)
> ERROR: invalid regular expression: parentheses () not balanced
>
> It's sending the server a bogus pattern in the second case.
> I've not looked at the code yet, but this does seem like a
> psql (not server) bug.
>
> As Adrian noted, you're not supposed to use parameter names
> in \df, only their types. It seems like whitespace ought
> to be allowed though.
>
> regards, tom lane
Thank you. Much relieved.

The no-space-no-names thing is suspicious. Away from desk just now but sure I’ve successfully cutpasted the arg list into \df calls Will test shortly

In response to

Browse pgsql-general by date

  From Date Subject
Next Message Adrian Klaver 2022-04-15 20:22:07 Re: function defined (or not), more worries on version 10->14 upgrade
Previous Message Tom Lane 2022-04-15 17:14:19 Re: function defined (or not), more worries on version 10->14 upgrade