Re: jsonb generator functions

On 12/08/2014 01:00 PM, Andrew Dunstan wrote:
>
> On 12/08/2014 04:21 AM, Alvaro Herrera wrote:
>> Andrew Dunstan wrote:
>>
>>> OK, here is a new patch version that
>>>
>>>   * uses find_coercion_path() to find the cast function if any, as
>>>     discussed elsewhere
>>>   * removes calls to getTypeOutputInfo() except where required
>>>   * honors a cast to json only for rendering both json and jsonb
>>>   * adds processing for the date type that was previously missing in
>>>     datum_to_jsonb
>> Did this go anywhere?
>>
>
> Not, yet. I hope to get to it this week.
>
>


OK, here is a new version.

The major change is that the aggregate final functions now clone the 
transition value rather than modifying it directly, avoiding a similar 
nearby error which Tom fixed recently.

Also here is a patch factored out which applies the 
find_coercion_pathway change to json.c. I'm inclined to say we should 
backpatch this to 9.4 (and with a small change 9.3). Thoughts?

cheers

andrew