Re: [PATCHES] libpq type system 0.9a

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

In response to

Browse pgsql-hackers by date

  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

Browse pgsql-patches by date

  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