]> granicus.if.org Git - postgresql/commit
Don't set reachedMinRecoveryPoint during crash recovery. In crash recovery,
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 9 Dec 2011 12:32:42 +0000 (14:32 +0200)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Fri, 9 Dec 2011 13:30:23 +0000 (15:30 +0200)
commite45057e0a7c4f5c6f1d662853c616eea26e9b0e4
treeeca1c2045637095b21821ba10475c392f22a4af7
parent85d85ff7ef04510f24fe5897e5535d765d7a293c
Don't set reachedMinRecoveryPoint during crash recovery. In crash recovery,
we don't reach consistency before replaying all of the WAL. Rename the
variable to reachedConsistency, to make its intention clearer.

In master, that was an active bug because of the recent patch to
immediately PANIC if a reference to a missing page is found in WAL after
reaching consistency, as Tom Lane's test case demonstrated. In 9.1 and 9.0,
the only consequence was a misleading "consistent recovery state reached at
%X/%X" message in the log at the beginning of crash recovery (the database
is not consistent at that point yet). In 8.4, the log message was not
printed in crash recovery, even though there was a similar
reachedMinRecoveryPoint local variable that was also set early. So,
backpatch to 9.1 and 9.0.
src/backend/access/transam/xlog.c