git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

Make crash recovery ignore recovery target settings.

author Fujii Masao <fujii@postgresql.org>

2019年9月30日 01:18:15 +0000 (10:18 +0900)

committer Fujii Masao <fujii@postgresql.org>

2019年9月30日 01:18:15 +0000 (10:18 +0900)

In v11 or before, recovery target settings could not take effect in
crash recovery because they are specified in recovery.conf and
crash recovery always starts without recovery.conf. But commit
2dedf4d9a8 integrated recovery.conf into postgresql.conf and
which unexpectedly allowed recovery target settings to take effect
even in crash recovery. This is definitely not good behavior.

To fix the issue, this commit makes crash recovery always ignore
recovery target settings.

Back-patch to v12.

Author: Peter Eisentraut
Reviewed-by: Fujii Masao
Discussion: https://postgr.es/m/e445616d-023e-a268-8aa1-67b8b335340c@pgmasters.net

src/backend/access/transam/xlog.c patch | blob | blame | history

diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c

index 6c69eb6dd76dce2243dae82c56cb077b68091d9b..61ba6b852e6623f3065cfc0404a88605ed2d3fea 100644 (file)

--- a/src/backend/access/transam/xlog.c

+++ b/src/backend/access/transam/xlog.c

@@ -5618,6 +5618,13 @@ recoveryStopsBefore(XLogReaderState *record)

TimestampTz recordXtime = 0;

TransactionId recordXid;

+ /*

+ * Ignore recovery target settings when not in archive recovery (meaning

+ * we are in crash recovery).

+ */