Re: Inadequate thought about buffer locking during hot standby replay - Mailing list pgsql-hackers

From Simon Riggs
Subject Re: Inadequate thought about buffer locking during hot standby replay
Date
Msg-id CA+U5nMLjiR_+32KX_F46heLDaFyZtPzCtKDFRoKNqLBMhZuWTg@mail.gmail.com
Whole thread Raw
In response to Re: Inadequate thought about buffer locking during hot standby replay  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On 12 November 2012 14:51, Tom Lane <tgl@sss.pgh.pa.us> wrote:

>> The 9.0 code is broken, however. In
>> 9.0, when a child page is split, the parent and new children are kept
>> locked until the downlinks are inserted/updated. If a concurrent scan
>> comes along and sees that incomplete state, it will miss tuples on the
>> new right siblings. We rely on a rm_cleanup operation at the end of WAL
>> replay to fix that situation, if the downlink insertion record is not
>> there. I don't see any easy way to fix that, unfortunately. Perhaps we
>> could backpatch the 9.1 rewrite, now that it's gotten some real-world
>> testing, but it was a big change so I don't feel very comfortable doing
>> that.
>
> Me either.  Given the lack of field complaints, I think we're better
> advised to just leave it unfixed in 9.0.  It'd not be a step forward
> if we broke something trying to make this work.

Agreed. Most people running 9.0 with GIST indexes have already upgraded.

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



pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: Inadequate thought about buffer locking during hot standby replay
Next
From: Markus Wanner
Date:
Subject: Re: Enabling Checksums