Thread: [HACKERS] Json support for array pseudotypes
The attached tiny patch lets the to_json(b) routines convert objects with array pseudotypes to json{b}. The main impetus for this came from my trying to convert the annyarray columns in pg_stats to json and finding that they ended up as text instead of json arrays. This way to_json on these columns does what is expected. cheers andrew -- Andrew Dunstan https://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Attachment
Andrew Dunstan <andrew.dunstan@2ndquadrant.com> writes: > The attached tiny patch lets the to_json(b) routines convert objects > with array pseudotypes to json{b}. The main impetus for this came from > my trying to convert the annyarray columns in pg_stats to json and > finding that they ended up as text instead of json arrays. This way > to_json on these columns does what is expected. Looks reasonable offhand, but maybe you should add some test cases? regards, tom lane
On 02/21/2017 06:01 PM, Tom Lane wrote: > Andrew Dunstan <andrew.dunstan@2ndquadrant.com> writes: >> The attached tiny patch lets the to_json(b) routines convert objects >> with array pseudotypes to json{b}. The main impetus for this came from >> my trying to convert the annyarray columns in pg_stats to json and >> finding that they ended up as text instead of json arrays. This way >> to_json on these columns does what is expected. > Looks reasonable offhand, but maybe you should add some test cases? > > Yeah, I will. Not sure how to add a test for the RECORDARRAYOID case. We don't seem to have any of those hanging around. cheers andrew -- Andrew Dunstan https://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services