From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Andrew Chernow <ac(at)esilo(dot)com>, James Mansion <james(at)mansionfamily(dot)plus(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: libpq WSACleanup is not needed |
Date: | 2009-01-20 15:09:51 |
Message-ID: | 4975E93F.6070703@hagander.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Bruce Momjian wrote:
> Andrew Chernow wrote:
>> Bruce Momjian wrote:
>>> We could have gone with a more elegant init/uninit solution but there is
>>> a history of slow upstream adoption of libpq API changes.
>>>
>>>
>> If that's the case, adding a connectdb option seems like a good
>> alternative. Orignally suggested here:
>>
>> http://archives.postgresql.org/pgsql-hackers/2009-01/msg01358.php
>
> Right, well the big question is how many people are going to use the
> connection option vs. doing it for everyone automatically.
>
> One possible approach might be to do it automatically, and allow a
> connection option to disable the WSACleanup() call.
I think that was the suggestion. Have an option that would disable
*both* the startup and the cleanup call, leaving the responsibility to
the app.
You can do this for SSL today by calling PQinitSSL().
> Actually, right now, if you have two libpq connections, and close one,
> does WSACleanup() get called, and does it affect the existing
> connection?
WSACleanup() gets called, but it has an internal reference count so it
does not have any effect on existing connections.
//Magnus
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2009-01-20 15:17:23 | Re: FWD: Re: Updated backslash consistency patch |
Previous Message | Magnus Hagander | 2009-01-20 14:51:16 | Re: [BUGS] BUG #4186: set lc_messages does not work |