Re: [PERFORM] [pgsql-hackers-win32] scalability issues on win32

Attached patch solves the problem for me.Didn't see anything of the
problem you described about hangs in my implementation. Probably because
mine exits at the first sign of a linebreak.

//Magnus

>-----Original Message-----
>From: pgsql-hackers-win32-owner@postgresql.org
>[mailto:pgsql-hackers-win32-owner@postgresql.org] On Behalf Of
>Claudio Natoli
>Sent: den 17 december 2004 01:31
>To: Bruce Momjian
>Cc: Merlin Moncure; Tom Lane; Win 32 hackers PGSQL
>Subject: Re: [PERFORM] [pgsql-hackers-win32] scalability
>issues on win32
>
>
>
>Hi Bruce and all,
>
>sorry guys, I won't have a chance to get to this until after
>the Christmas break.
>
>If one of the other usual suspects (Dave, Andrew, Magnus,
>Merlin) can claim it sooner, please do!
>
>Cheers,
>Claudio
>
>
>
>> -----Original Message-----
>> From: Bruce Momjian [mailto:pgman@candle.pha.pa.us]
>> Sent: Friday, 17 December 2004 11:25 AM
>> To: Claudio Natoli
>> Cc: 'Merlin Moncure'; Tom Lane; Win 32 hackers PGSQL
>> Subject: Re: [PERFORM] [pgsql-hackers-win32] scalability
>> issues on win32
>>
>>
>>
>> Claudio, I need this completed so we can get it into CVS.
>> Without it a
>> 8.0.0 version string will fail to compare.
>>
>> --------------------------------------------------------------
>> -------------
>>
>> Claudio Natoli wrote:
>> >
>> > Hi Merlin,
>> >
>> > > On a probably unrelated note:
>> > > Can some other win32 hacker try the following?
>> > > Download the latest cvs sources from the server and edit
>> configure,
>> > > replacing all instances of '8.0beta5' with '8.0' etc. as is
>> > > will be when released.  Following that do a full
>> make/make clean.  After
>> > > that try and start he server with pg_ctl.exe and tell me
>> if it succeeds or
>> > not.
>> >
>> > I can confirm this failure.
>> >
>> > (in my case at least) pg_ctl is not seeing the full version
>> string of the
>> > postmaster in find_other_exec. This is due to the single
>> call to ReadFile in
>> > pipe_read_line, which is not returning the entire version
>> string (losing all
>> > characters beyond the last decimal point in the version string). A
>> > subsequent call to ReadFile picks up the remainder of the
>> string. I don't
>> > see that we can rely on a single call returning the entire
>> version string.
>> >
>> > If someone can run with this, I've attached initial looping
>> changes for
>> > pipe_read_line. However I'm not sure of the correct
>> terminating condition
>> > (gets into a call to ReadFile that never returns; the
>> WaitForSingleObject
>> > appears insufficient as a precondition for ReadFile) and
>> just don't have
>> > time to complete it right now.
>> >
>> > Cheers,
>> > Claudio
>> >