Make recovery report error message when invalid page header is found.
authorFujii Masao <fujii@postgresql.org>
Tue, 5 Oct 2021 15:16:03 +0000 (00:16 +0900)
committerFujii Masao <fujii@postgresql.org>
Tue, 5 Oct 2021 15:16:03 +0000 (00:16 +0900)
commit68601985e699adeb267636fd19d3d6113554bd1f
tree147e985ef86d3c3ccbc601005a4ae256d084f932
parentf3fec23dbdead113700fb1b401b681fa24f1e4f4
Make recovery report error message when invalid page header is found.

Commit 0668719801 changed XLogPageRead() so that it validated the page
header, if invalid page header was found reset the error message and
retried reading the page, to fix the scenario where streaming standby
got stuck at a continuation record. This change hid the error message
about invalid page header, which would make it harder for users to
investigate what the actual issue was found in WAL.

To fix the issue, this commit makes XLogPageRead() report the error
message when invalid page header is found.

When not in standby mode, an invalid page header should cause recovery
to end, not retry reading the page, so XLogPageRead() doesn't need to
validate the page header for the retry. Instead, ReadPageInternal() should
be responsible for the validation in that case. Therefore this commit
changes XLogPageRead() so that if not in standby mode it doesn't validate
the page header for the retry.

Reported-by: Yugo Nagata
Author: Yugo Nagata, Kyotaro Horiguchi
Reviewed-by: Ranier Vilela, Fujii Masao
Discussion: https://postgr.es/m/20210718045505.32f463ed6c227111038d8ae4@sraoss.co.jp
src/backend/access/transam/xlog.c