Andrew Dunstan <[email protected]> writes: > Maybe we need to look at all the places we call GetLastError(). There > are quite a few of them.
It would only be an issue with syscalls that have badly designed APIs like this one. Most of the time you know that the function has failed and is supposed to have set the error code. What I'm wondering about right now is whether the sleep-and-retry logic is needed at all, if we get the error code handling straight. A look in the archives says that Magnus added it between these two versions of his draft patch: http://archives.postgresql.org//pgsql-patches/2007-03/msg00250.php http://archives.postgresql.org//pgsql-patches/2007-03/msg00263.php without any indication of why, except that I had complained that some error checks seemed to be missing in the original. I wonder if it was a misguided attempt to fix a stale-error-code problem. regards, tom lane -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
