From: | "Merlin Moncure" <mmoncure(at)gmail(dot)com> |
---|---|
To: | "Andrew Dunstan" <andrew(at)dunslane(dot)net> |
Cc: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Greg Sabino Mullane" <greg(at)turnstep(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [PATCHES] libpq type system 0.9a |
Date: | 2008-04-08 18:53:23 |
Message-ID: | b42b73150804081153y67cfeb69y6f60604f45b9449@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
On Tue, Apr 8, 2008 at 2:49 PM, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
> > Better support for arrays and composites is certainly something that
> > people might want, but the problem with this design is that it forces
> > them to buy into a number of other decisions that they don't necessarily
> > want.
> >
> > I could see adding four functions to libpq that create and parse
> > the textual representations of arrays and records.
> Well, that was the part that interested me, so let me now speak up in favor
> of better array/record support in libpq.
by the way, we handle both text and binary array results...and getting
things in binary is _much_ faster. not to mention text is
destructive. for example, composite types in text do not return the
oid of composite member fields.
with our patch, since you can 'pop' a result of a returned composite,
or array of composite, you have access to all that information in the
result api. so I would argue that allowing text only parsing only
recovers a portion of the provided functionality.
merlin
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Chernow | 2008-04-08 18:55:30 | Re: [PATCHES] libpq type system 0.9a |
Previous Message | Heikki Linnakangas | 2008-04-08 18:52:01 | Re: Free Space Map data structure |
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Chernow | 2008-04-08 18:55:30 | Re: [PATCHES] libpq type system 0.9a |
Previous Message | Andrew Dunstan | 2008-04-08 18:49:54 | Re: [PATCHES] libpq type system 0.9a |