From fb47544d0ca18312bf0b9e53e335217db5c41a08 Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Tue, 14 Feb 2017 12:41:01 -0500 Subject: Minor fixes for WAL consistency checking. Michael Paquier, reviewed and slightly revised by me. Discussion: http://postgr.es/m/CAB7nPqRzCQb=vdfHvMtP0HMLBHU6z1aGdo4GJsUP-HP8jx+Pkw@mail.gmail.com --- src/backend/access/transam/xlog.c | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) (limited to 'src/backend/access/transam/xlog.c') diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c index 2dcff7f54b5..f23e1086280 100644 --- a/src/backend/access/transam/xlog.c +++ b/src/backend/access/transam/xlog.c @@ -1360,6 +1360,16 @@ checkXLogConsistency(XLogReaderState *record) Assert(XLogRecHasBlockImage(record, block_id)); + if (XLogRecBlockImageApply(record, block_id)) + { + /* + * WAL record has already applied the page, so bypass the + * consistency check as that would result in comparing the full + * page stored in the record with itself. + */ + continue; + } + /* * Read the contents from the current buffer and store it in a * temporary page. @@ -1390,7 +1400,7 @@ checkXLogConsistency(XLogReaderState *record) /* * Read the contents from the backup copy, stored in WAL record and - * store it in a temporary page. There is not need to allocate a new + * store it in a temporary page. There is no need to allocate a new * page here, a local buffer is fine to hold its contents and a mask * can be directly applied on it. */ -- cgit v1.2.3