|From:||"Dave Page" <dpage(at)vale-housing(dot)co(dot)uk>|
|To:||"Anoop Kumar" <anoopk(at)pervasive-postgres(dot)com>|
|Subject:||Re: Leak repairs|
|Views:||Raw Message | Whole Thread | Download mbox|
> -----Original Message-----
> From: Anoop Kumar [mailto:anoopk(at)pervasive-postgres(dot)com]
> Sent: 19 July 2005 07:35
> To: Dave Page
> Cc: pgsql-odbc(at)postgresql(dot)org
> Subject: RE: [ODBC] Leak repairs
> Hi Dave,
> I have tested the patch from you, and it does make the perfmon graph
> less steep. I am applying the patch to CVS.
Great, thanks. Any joy tracking down the last little bit?
> Regarding the call to PGgetisnull: My mistake! I forgot to remove it,
> thanks for pointing it out.
I think we should put out a second snapshot sometime, as soon as the
rest of this memory leak is fixed, and the Excel problem is sorted. Any
progress on that one?
Also we need to expose the libpq SSL option to the user, probably on the
main connection dialogue, and obviously as a connection string option.
At the same time, the protocol version options can be removed. I can
probably look at that if you are pushed for time - let me know what you
Finally, the threading problem. After discussion with Bruce Momjian and
Magnus Hagander, we think that the --enable-thread-safety configure
option should be used on Windows as well as other platforms as it does
do more than just enable critical sections around non-thread safe API
calls. I am (as time allows!) working on fixing PostgreSQL to allow
|Next Message||Marko Ristola||2005-07-19 07:36:48||Re: Leak repairs|
|Previous Message||Anoop Kumar||2005-07-19 06:35:09||Re: Leak repairs|