Re: BUG #14849: jsonb_build_object doesn't like VARIADIC calls very much

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>
Cc: Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Marko Tiikkaja <marko(at)joh(dot)to>, PostgreSQL mailing lists <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: BUG #14849: jsonb_build_object doesn't like VARIADIC calls very much
Date: 2017-10-21 03:40:39
Message-ID: CAB7nPqRkQ9YCiXfT3Ucn7XUx_yfmAsRTKxewfPGbaS1GczLnbw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On Fri, Oct 20, 2017 at 10:49 PM, Andrew Dunstan
<andrew(dot)dunstan(at)2ndquadrant(dot)com> wrote:
>
>
> On 10/19/2017 11:25 AM, Michael Paquier wrote:
>> On Thu, Oct 19, 2017 at 10:13 PM, Andrew Dunstan
>> <andrew(dot)dunstan(at)2ndquadrant(dot)com> wrote:
>>>
>>> On 10/18/2017 10:07 PM, Michael Paquier wrote:
>>>> Please see the attached patch which considers all the discussion done,
>>>> which shapes the code in the way I see most suited for HEAD and
>>>> back-branches.
>>>
>>> Patch doesn't apply against HEAD.
>>>
>>> Also, don't we need more tests?
>> Oops. Sorry about that. v3 is incorrect, I messed up the origin commit
>> for the diffs generated. Here you go as attached.
>
>
> Looks good. Do you want to produce patches for the back branches also?
> this applies to REL_10 but not earlier. If not I'll work on it this weekend.

Here you go, down to 9.4 where things apply. The same patch can be
applied for HEAD/10, 9.5/9.6 and 9.4. The 9.4 version just needs to
have the portion for json as the jsonb functions have been added in
9.5, and the json functions are new as of 9.4.
--
Michael

Attachment Content-Type Size
json_variadic_v5_master_10.patch text/x-patch 26.9 KB
json_variadic_v5_96_95.patch text/x-patch 26.9 KB
json_variadic_v5_94.patch text/x-patch 13.1 KB

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2017-10-21 15:43:42 Re: BUG #14849: jsonb_build_object doesn't like VARIADIC calls very much
Previous Message Andrew Dunstan 2017-10-20 13:49:13 Re: BUG #14849: jsonb_build_object doesn't like VARIADIC calls very much

Browse pgsql-hackers by date

  From Date Subject
Next Message Haribabu Kommi 2017-10-21 05:30:38 Re: Refactor handling of database attributes between pg_dump and pg_dumpall
Previous Message Peter Geoghegan 2017-10-21 00:34:18 Re: A design for amcheck heapam verification