From: | Jeff Davis <pgsql(at)j-davis(dot)com> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>, Merlin Moncure <mmoncure(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: crash-safe visibility map, take five |
Date: | 2011-06-23 00:53:54 |
Message-ID: | 1308790434.4717.93.camel@jdavis-ux.asterdata.local |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, 2011-06-16 at 23:17 -0400, Noah Misch wrote:
> 2. In the words of a comment added by the patch:
> * The critical integrity requirement here is that we must never end up with
> * a situation where the visibility map bit is set, and the page-level
> * PD_ALL_VISIBLE bit is clear. If that were to occur, then a subsequent
> * page modification would fail to clear the visibility map bit.
> It does this by WAL-logging the operation of setting a vm bit. This also has
> the benefit of getting vm bits set correctly on standbys.
In the same function, there is also the comment:
"We don't bump the LSN of the heap page when setting the visibility
map bit, because that would generate an unworkable volume of
full-page writes. This exposes us to torn page hazards, but since
we're not inspecting the existing page contents in any way, we
don't care."
It would be nice to have a comment explaining why that is safe with
respect to the WAL-before-data rule. Obviously torn pages aren't much of
a problem, because it's a single bit and completely idempotent.
Regards,
Jeff Davis
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2011-06-23 01:53:46 | Re: crash-safe visibility map, take five |
Previous Message | Peter Geoghegan | 2011-06-23 00:06:43 | Re: Coding style point: "const" in function parameter declarations |