Commit072521b

committed

Rely only on checkpoint 1 at end of recovery.

Searching for checkpoint 2 (previous) is notcorrect in all cases.Bug report from Heikki Linnakangas

1 parente1c1e21 commit072521bCopy full SHA for 072521b

File tree

-1

lines changed

-1

lines changed

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -5777,7 +5777,13 @@ StartupXLOG(void)`
`5777`	`5777`	`if (fast_promote)`
`5778`	`5778`	`{`
`5779`	`5779`	`checkPointLoc=ControlFile->prevCheckPoint;`
`5780`		`-record=ReadCheckpointRecord(xlogreader,checkPointLoc,2, false);`
	`5780`	`+`
	`5781`	`+/*`
	`5782`	`+ * Confirm the last checkpoint is available for us to recover`
	`5783`	`+ * from if we fail. Note that we don't check for the secondary`
	`5784`	`+ * checkpoint since that isn't available in most base backups.`
	`5785`	`+ */`
	`5786`	`+record=ReadCheckpointRecord(xlogreader,checkPointLoc,1, false);`
`5781`	`5787`	`if (record!=NULL)`
`5782`	`5788`	`{`
`5783`	`5789`	`checkpoint_wait= false;`

Comments

(0)