Magnus Hagander wrote:
> Tom Lane wrote:
>
>> Andrew Dunstan <andrew@dunslane.net> writes:
>>
>>> Now presumably we sleep for 1 sec between the CloseHandle() call and the
>>> CreateFileMapping() call in that code for a reason.
>>>
>> I'm not sure. Magnus never did answer my question about why the sleep
>> and retry was put in at all; it seems not unlikely from here that it
>> was mere speculation.
>>
>
> It was necessary at the time.
>
> The actual 1 second value was completely random - it fixed all the
> issues on my test VM at the time. I don't recall exactly the details,
> but I do recall having to run a lot of tests before I managed to provoke
> an error, and that with the 1 sec thing i could run it for a day of
> repeated restarts without any errors.
>
>
>
Well, my untested hypothesis is that the actual time required is
variable, depending on environmental factors such as machine load. So
testing repeatedly where such factors are constant might not be good
enough. That's why I suggested some sort of increasing backoff, in an
attempt to be adaptive.
cheers
andrew