Hello Stephen,
I certainly don't see a lot of point in doing much more than what was
discussed previously for 'new' blocks (counting them as skipped and
moving on).
Sure.
An actual read() error (that is, a failure on a read() call such as
getting back EIO), on the other hand, is something which I'd probably
report back to the user immediately and then move on, and perhaps
report again at the end.
Yep.
Note that a short read isn't an error and falls under the 'new' blocks
discussion above.
I'm really unsure that a short read should really be coldly skipped:
If the check is offline, then one file is in a very bad state, this is
really a panic situation.
If the check is online, given that both postgres and the verify command
interact with the same OS (?) and at the pg page level, I'm not sure in
which situation there could be a partial block, because pg would only
send full pages to the OS.
--
Fabien.
