Heikki Linnakangas <heikki.linnakangas@enterprisedb.com> writes:
> Hmm. I think an enum is better than a bitmask here. At the moment, we need
> three different modes of operation:
> 1. Read the page as usual, throw an error on corrupted page (ReadBuffer())
> 2. Read the page, zero page on corruption (this is new)
Is this new? Would it make sense for zero_damaged_pages to use this? Perhaps
the enum should have an option to error on damaged pages, warn and zero
damaged pages, or just zero damaged pages.
We might also want different behaviour for pages for which the crc doesn't
match versus pages that have nonsensical page headers.
> 3. Don't read the page from disk, just allocate a buffer. (ReadOrZeroBuffer())
-- Gregory Stark EnterpriseDB http://www.enterprisedb.com Ask me about EnterpriseDB's Slony Replication
support!
