Re: visibilitymap_clear()s in vacuumlazy.c aren't WAL logged - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: visibilitymap_clear()s in vacuumlazy.c aren't WAL logged
Date
Msg-id CAB7nPqQLCu60LCmF9_+jofa3oVOqLFnKfckJD0HZzjNz86Y97w@mail.gmail.com
Whole thread Raw
In response to visibilitymap_clear()s in vacuumlazy.c aren't WAL logged  (Andres Freund <andres@anarazel.de>)
List pgsql-hackers
On Sat, Jul 16, 2016 at 10:23 AM, Andres Freund <andres@anarazel.de> wrote:
> The $subject says it all. Am I missing something, or is that not ok?

Indeed, it would be a good thing to get those sanity checks logged so
as standbys get the call, or at least perform the sanity check as well
on the block that the system is warning about. It seems to me that
logging the VM bit would be an interesting option as well, bringing to
the fact that there should be dedicated WAL records.

(By the way, adding a check for the case where a page contains
non-frozen tuples but the VM has the all-frozen bit set would be good
as well)

> Now, these branches should never be hit, but it surely isn't good that
> the corruption will persist on a primary, after it's explicitly been
> fixed on the standby.

"primary" and "standby" are reversed here?
-- 
Michael



pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Reviewing freeze map code
Next
From: Michael Paquier
Date:
Subject: Re: Regression tests vs existing users in an installation