crash-safe visibility map, take five - Mailing list pgsql-hackers

From Robert Haas
Subject crash-safe visibility map, take five
Date
Msg-id BANLkTimuLk4RHXSQHEEiYGbxiXp2mh5KCA@mail.gmail.com
Whole thread Raw
Responses Re: crash-safe visibility map, take five  (Rob Wultsch <wultsch@gmail.com>)
Re: crash-safe visibility map, take five  (Merlin Moncure <mmoncure@gmail.com>)
List pgsql-hackers
On Wed, Mar 30, 2011 at 8:52 AM, Heikki Linnakangas
<heikki.linnakangas@enterprisedb.com> wrote:
>> Another question:
>> To address the problem in
>> http://archives.postgresql.org/pgsql-hackers/2010-02/msg02097.php
>> , should we just clear the vm before the log of insert/update/delete?
>> This may reduce the performance, is there another solution?
>
> Yeah, that's a straightforward way to fix it. I don't think the performance
> hit will be too bad. But we need to be careful not to hold locks while doing
> I/O, which might require some rearrangement of the code. We might want to do
> a similar dance that we do in vacuum, and call visibilitymap_pin first, then
> lock and update the heap page, and then set the VM bit while holding the
> lock on the heap page.

Here's an attempt at implementing the necessary gymnastics.

Comments?

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment

pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: Prefered Types
Next
From: Tom Lane
Date:
Subject: Re: Why not install pgstattuple by default?