On Tue, May 13, 2008 at 2:38 AM, Tom Lane <[EMAIL PROTECTED]> wrote: > Andrew Dunstan <[EMAIL PROTECTED]> writes: > > > Simon Riggs wrote: > >> Well, the patch was rejected long ago, not sure why its in this > >> commitfest. But its an open issue on the Windows port. > > > Surely the right fix is to use the recently implemented > > pgwin32_safestat() (if we aren't already - I suspect we probably are) > > and remove the kluge in pg_standby.c. > > I think the open issue is how to know whether pgwin32_safestat fixes the > problem that the kluge tried to work around.
Per the comments on the commitfest page, I don't believe it is. pgwin32_safestat fixes a bug in which stat() returns stale information (if memory serves). The hack in pg_standby was added because copy in Windows appears to preallocate the required space for the file it's copying, thus checking the file size to verify that the copy has completed is not a valid test. -- Dave Page EnterpriseDB UK: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
