Re: race condition in sync rep - Mailing list pgsql-hackers

From Simon Riggs
Subject Re: race condition in sync rep
Date
Msg-id AANLkTimoTHQyO4ZpE50Lo+9QJ5Gs6SWmX=o9nfB3oWty@mail.gmail.com
Whole thread Raw
In response to Re: race condition in sync rep  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: race condition in sync rep
List pgsql-hackers
On Sat, Mar 26, 2011 at 4:04 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>>> I've added code for you.
>
>> Your skepticism seems out of place. I actually hit this problem in testing. We could debate how realistic my test
setupwas, but why?  It is not our policy - and bad practice in general - to ship code with race conditions.  The code
youadded is not "for me"; it's "to fix a race condition". 
>
> In particular, in view of today's fix, shouldn't this commit be reverted?
>
> http://git.postgresql.org/gitweb?p=postgresql.git;a=commitdiff;h=6e8e7cc580665ddd43c8ca2acc6d60f345570a57
>
> I thought at the time that that was nothing more than documenting a
> known bug, and now it is documenting a dead bug.

No, that doc change is still accurate.

--
 Simon Riggs                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services


pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: Lock problem with autovacuum truncating heap
Next
From: Tom Lane
Date:
Subject: Re: race condition in sync rep