database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-01 01:04:50 +03:00
Code

Fix pause_at_recovery_target + recovery_target_inclusive combination.

Browse Source 
If pause_at_recovery_target is set, recovery pauses *before* applying the
target record, even if recovery_target_inclusive is set. If you then
continue with pg_xlog_replay_resume(), it will apply the target record
before ending recovery. In other words, if you log in while it's paused
and verify that the database looks OK, ending recovery changes its state
again, possibly destroying data that you were tring to salvage with PITR.

Backpatch to 9.1, this has been broken since pause_at_recovery_target was
added.
This commit is contained in:
Heikki Linnakangas 2014-01-08 23:06:03 +02:00
parent 82c75f9dd3
commit 3bd8987ef8
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/backend/access/transam/xlog.c
View File

@ -6868,11 +6868,6 @@ StartupXLOG(void)
*/
if (recoveryStopsHere(record, &recoveryApply))
{
if (recoveryPauseAtTarget)
{
SetRecoveryPause(true);
recoveryPausesHere();
}
reachedStopPoint = true; /* see below */
recoveryContinue = false;
@ -6951,6 +6946,12 @@ StartupXLOG(void)
* end of main redo apply loop
*/
if (recoveryPauseAtTarget && reachedStopPoint)
{
SetRecoveryPause(true);
recoveryPausesHere();
}
ereport(LOG,
(errmsg("redo done at %X/%X",
ReadRecPtr.xlogid, ReadRecPtr.xrecoff)));