NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitd56b629

committed

Must not reach consistency before XLOG_BACKUP_RECORD

When waiting for an XLOG_BACKUP_RECORD the minRecoveryPointwill be incorrect, so we must not declare recovery as consistentbefore we have seen the record. Major bug allowing recovery to endtoo early in some cases, allowing people to see inconsistent db.This patch to HEAD and 9.2, other fix required for 9.1 and 9.0Simon Riggs and Andres Freund, bug report by Jeff Janes

1 parent0811f4f commitd56b629Copy full SHA for d56b629

File tree

1 file changed

-2

lines changed

src/backend/access/transam
- xlog.c

1 file changed

-2

lines changed

`‎src/backend/access/transam/xlog.c`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -7155,9 +7155,12 @@ CheckRecoveryConsistency(void)`
`7155`	`7155`	`return;`
`7156`	`7156`
`7157`	`7157`	`/*`
`7158`		`- * Have we passed our safe starting point?`
	`7158`	`+ * Have we passed our safe starting point? Note that minRecoveryPoint`
	`7159`	`+ * is known to be incorrectly set if ControlFile->backupEndRequired,`
	`7160`	`+ * until the XLOG_BACKUP_RECORD arrives to advise us of the correct`
	`7161`	`+ * minRecoveryPoint. All we prior to that is its not consistent yet.`
`7159`	`7162`	`*/`
`7160`		`-if (!reachedConsistency&&`
	`7163`	`+if (!reachedConsistency&& !ControlFile->backupEndRequired&&`
`7161`	`7164`	`XLByteLE(minRecoveryPoint,EndRecPtr)&&`
`7162`	`7165`	`XLogRecPtrIsInvalid(ControlFile->backupStartPoint))`
`7163`	`7166`	`{`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitd56b629

File tree

1 file changed

1 file changed

`‎src/backend/access/transam/xlog.c`

0 commit comments