Re: Adding argument names to aggregate functions - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: Adding argument names to aggregate functions
Date
Msg-id 98834F2E-B4E3-4084-8D81-4B3DD8D666E0@yesql.se
Whole thread Raw
In response to Re: Adding argument names to aggregate functions  (Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>)
Responses Re: Adding argument names to aggregate functions
List pgsql-hackers
> On 19 Jul 2023, at 19:32, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:
>
> Daniel Gustafsson <daniel@yesql.se> writes:
>
>> This patch no longer applied but had a fairly trivial conflict so I've attached
>> a rebased v3 addressing the conflict in the hopes of getting this further.
>
> Thanks for the heads-up!  Turns out the conflict was due to the new
> json(b)_object_agg(_unique)(_strict) functions, which should also have
> proargnames added.  Here's an updated patch that does that.

Great, thanks!  I had a quick look at this while rebasing (as well as your
updated patch) and it seems like a good idea to add this.  Unless there are
objections I will look at getting this in.

--
Daniel Gustafsson




pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: Extracting cross-version-upgrade knowledge from buildfarm client
Next
From: Daniel Gustafsson
Date:
Subject: Re: Giving more detail in pg_upgrade errormessage