Horton

Re: PATCH: recursive json_populate_record()

David Steele <david@pgmasters.net>

View thread View on postgr.es
From: David Steele <david@pgmasters.net>
To: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
Cc: Nikita Glukhov <n.gluhov@postgrespro.ru>, Michael Paquier <michael.paquier@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, Aleksander Alekseev <a.alekseev@postgrespro.ru>, PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Date: 2017-03-21T21:26:54Z
Lists: pgsql-hackers 
On 3/21/17 2:31 PM, Andrew Dunstan wrote:
> On 03/21/2017 01:37 PM, David Steele wrote:
 >>
>> This thread has been idle for months since Tom's review.
>>
>> The submission has been marked "Returned with Feedback".  Please feel
>> free to resubmit to a future commitfest.
>
> Please revive. I am planning to look at this later this week.

Revived in "Waiting on Author" state.

-- 
-David
david@pgmasters.net

Commits

  1. Make edge-case behavior of jsonb_populate_record match json_populate_record

  2. Fix thinko in JsObjectSize() macro.

  3. Introduce convenience macros to hide JsonbContainer header accesses better.