I wrote: > On Sun, Jun 14, 2009 at 1:04 PM, Robert Haas<robertmh...@gmail.com> wrote: >> On Sun, Jun 14, 2009 at 1:02 PM, Tom Lane<t...@sss.pgh.pa.us> wrote: >>> Robert Haas <robertmh...@gmail.com> writes: >>>> On Sun, Jun 14, 2009 at 11:28 AM, Tom Lane<t...@sss.pgh.pa.us> wrote: >>>>> However, using BuildTupleFromCStrings is wasteful/stupid for *both* >>>>> text and xml output, so it seems like getting rid of it is the thing >>>>> to do here. >>> >>>> Makes sense. However, if we just make that change in do_tup_output(), >>>> then we'll break the ability to use that function for non-text >>>> datatypes. >>> >>> I'd envision it taking Datums, so it doesn't really matter. However, >>> as you say, specializing it to text only wouldn't be much of a loss. >> >> I like the Datum option, so I'll work up a patch for that, unless you >> want to just do it and spare me the trouble. :-) > > Here's an attempt. Is this anything like what you had in mind?
Hmm... on further review, I'm thinking this is still a bit wastful, because we don't really need (I think) to call TupleDescGetAttInMetadata from begin_tup_output_tupdesc. But I'm not sure what the best way is to avoid that. Any thoughts? ...Robert -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers