From: | "Dave Page" <dpage(at)vale-housing(dot)co(dot)uk> |
---|---|
To: | "Stephen Frost" <sfrost(at)snowman(dot)net>, "Martijn van Oosterhout" <kleptog(at)svana(dot)org> |
Cc: | <pgsql-hackers(at)postgresql(dot)org>, "Hiroshi Inoue" <inoue(at)tpf(dot)co(dot)jp> |
Subject: | Re: Practical impediment to supporting multiple SSL libraries |
Date: | 2006-04-13 13:07:19 |
Message-ID: | E7F85A1B5FF8D44C8A1AF6885BC9A0E4011C9CA4@ratbert.vale-housing.co.uk |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> -----Original Message-----
> From: Stephen Frost [mailto:sfrost(at)snowman(dot)net]
> Sent: 13 April 2006 14:03
> To: Martijn van Oosterhout
> Cc: Dave Page; pgsql-hackers(at)postgresql(dot)org; Hiroshi Inoue
> Subject: Re: [HACKERS] Practical impediment to supporting
> multiple SSL libraries
>
> * Martijn van Oosterhout (kleptog(at)svana(dot)org) wrote:
> > Well, I quickly scanned the code in CVS to see what I could
> find out.
>
> Wow, that was quick. :)
Yes :-)
> > I don't think the zero-copy is relevent, the code is not
> written in a
> > way that suggests speed was an issue. Rather I think the
> way you want
> > to use the resultset is the issue. You can't use the memory in the
> > PGresult because then'd you need to track which tuples were
> allocated
> > by you and which we allocated by libpq. The resulting copying is
> > needless, along with the fact that you double your memory usage.
>
> The double memory usage definitely sucks but I really think
> speed would also be greatly improved by removing the double
> copying and all the function calls dealing with PQgetResult, etc...
Don't forget that the code now in CVS-tip is not the code that had the
copy issue. As of last Saturday, the hybrid version was moved to tip.
Regards, Dave.
From | Date | Subject | |
---|---|---|---|
Next Message | Martijn van Oosterhout | 2006-04-13 13:07:56 | Re: Practical impediment to supporting multiple SSL libraries |
Previous Message | Stephen Frost | 2006-04-13 13:02:47 | Re: Practical impediment to supporting multiple SSL libraries |