Re: [HACKERS] json generation enhancements

2013-02-28 Thread Steve Singer
On 13-02-25 05:37 PM, Andrew Dunstan wrote: On 02/24/2013 01:09 AM, Steve Singer wrote: On 13-01-11 11:03 AM, Andrew Dunstan wrote: On 01/11/2013 11:00 AM, Andrew Dunstan wrote: I have not had anyone follow up on this, so I have added docs and will add this to the commitfest. Recap:

Re: [HACKERS] json generation enhancements

2013-02-25 Thread Andrew Dunstan
On 02/24/2013 01:09 AM, Steve Singer wrote: On 13-01-11 11:03 AM, Andrew Dunstan wrote: On 01/11/2013 11:00 AM, Andrew Dunstan wrote: I have not had anyone follow up on this, so I have added docs and will add this to the commitfest. Recap: This adds the following:

Re: [HACKERS] json generation enhancements

2013-02-24 Thread Andrew Dunstan
On 02/24/2013 02:15 AM, Craig Ringer wrote: On 02/24/2013 02:09 PM, Steve Singer wrote: Here is a review of this patch., Overview - This patch adds a set of functions to convert types to json. Specifically * An aggregate that take the elements and builds up a json array.

Re: [HACKERS] json generation enhancements

2013-02-24 Thread Craig Ringer
On 02/24/2013 09:58 PM, Andrew Dunstan wrote: Adding a cast to json for a builtin type will have no effect unless you also change this code. We can relax that but my view was that we should know how to generate JSON from builtin types and just do it. If json generation from built-in types is

Re: [HACKERS] json generation enhancements

2013-02-24 Thread Andrew Dunstan
On 02/24/2013 06:33 PM, Craig Ringer wrote: On 02/24/2013 09:58 PM, Andrew Dunstan wrote: Adding a cast to json for a builtin type will have no effect unless you also change this code. We can relax that but my view was that we should know how to generate JSON from builtin types and just do it.

Re: [HACKERS] json generation enhancements

2013-02-23 Thread Steve Singer
On 13-01-11 11:03 AM, Andrew Dunstan wrote: On 01/11/2013 11:00 AM, Andrew Dunstan wrote: I have not had anyone follow up on this, so I have added docs and will add this to the commitfest. Recap: This adds the following: json_agg(anyrecord) - json to_json(any) - json

Re: [HACKERS] json generation enhancements

2013-02-23 Thread Craig Ringer
On 02/24/2013 02:09 PM, Steve Singer wrote: Here is a review of this patch., Overview - This patch adds a set of functions to convert types to json. Specifically * An aggregate that take the elements and builds up a json array. * Conversions from an hstore to json For

[HACKERS] json generation enhancements

2013-01-11 Thread Andrew Dunstan
I have not had anyone follow up on this, so I have added docs and will add this to the commitfest. Recap: This adds the following: json_agg(anyrecord) - json to_json(any) - json hstore_to_json(hstore) - json (also used as a cast) hstore_to_json_loose(hstore) - json Also, in

Re: [HACKERS] json generation enhancements

2013-01-11 Thread Andrew Dunstan
On 01/11/2013 11:00 AM, Andrew Dunstan wrote: I have not had anyone follow up on this, so I have added docs and will add this to the commitfest. Recap: This adds the following: json_agg(anyrecord) - json to_json(any) - json hstore_to_json(hstore) - json (also used as a cast)