Re: pgstat: remove delayed destroy / pipe:

From: "Peter Brant" <Peter(dot)Brant(at)wicourts(dot)gov>
To: "Bruce Momjian" <pgman(at)candle(dot)pha(dot)pa(dot)us>, <pgsql-patches(at)postgresql(dot)org>
Cc: "Magnus Hagander" <mha(at)sollentuna(dot)net>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: pgstat: remove delayed destroy / pipe:
Date: 2006-05-01 17:15:25
Message-ID: 4455FBDD020000BE000034D1@gwmta.wicourts.gov
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

It turns out the problem is that port/pipe.c is compiled with -DFRONTEND
and include/port/win32.h wraps the recv to pgwin32_recv macro in a
#ifndef FRONTEND. We've actually been using the WinSock recv function
directly (verified with gcc -E).

If somebody else could take over actually fixing this, that would be
great. As I mentioned before, we're heading away from Windows for the
time being.

Pete

>>> Peter Brant 04/19/06 5:16 pm >>>
I'm still planning on doing this.

I did add a loop around the second WSARecv in pgwin32_recv() as that
was our best guess about where the error was coming from. As it turns
out, that's apparently not the case and I haven't had a chance to come
back to it yet.

It might be that my original patch (putting the loop in piperead()) is
the best approach. All indications are that it fixes the problem and
it's the lowest impact change. I'm still quite curious what API call is
returning the WSAEWOULDBLOCK though...

Pete

>>> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> 04/19/06 5:07 am >>>
Would someone generate a patch that includes all the new ideas and
post
it here? Thanks.

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Larry Rosenman 2006-05-01 18:15:13 Re: Logging pg_autovacuum
Previous Message Bruce Momjian 2006-05-01 02:58:39 Re: Patch for %Allow per-database permissions to be set via