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 |
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 |
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 |