Tom Lane wrote:
> Magnus Hagander <magnus@hagander.net> writes:
>> Passes my tests, but I can't really reproduce the requirement to retry,
>> so I haven't been able to test that part :(
>
> The patch looks sane to me. If you want to test, perhaps reducing the
> sleep to 1 msec or so would reproduce the need to go around the loop
> more than once. (Don't forget to put the machine under additional
> load, too.)
I've applied this to HEAD and 8.3 so we can get some buildfarm testing
on it as well.
Andrew, any chance you can get 8.3-tip tested with your client? Or at
least in your own reproducable-environment?
I didn't backpatch to 8.2, because the code is completely different
there. We should probably consider doing it once we know if this fixes
the actual issue, but I don't want to spend the effort on backporting it
until we know it works.
//Magnus