NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit64e196b

committed

Fix bufmgr so CHECKPOINT_END_OF_RECOVERY behaves as a shutdown checkpoint.

Recovery code documents clearly that a shutdown checkpoint is executed atend of recovery - a shutdown checkpoint WAL record is written but the buffermanager had been altered to treat end of recovery as a normal checkpoint.This bug exacerbates the bufmgr relpersistence bug.Bug spotted by Andres Freund, patch by me.

1 parent3275426 commit64e196bCopy full SHA for 64e196b

File tree

1 file changed

-2

lines changed

src/backend/storage/buffer
- bufmgr.c

1 file changed

-2

lines changed

`‎src/backend/storage/buffer/bufmgr.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1206,9 +1206,9 @@ BufferSync(int flags)`
`1206`	`1206`
`1207`	`1207`	`/*`
`1208`	`1208`	`* Unless this is a shutdown checkpoint, we write only permanent, dirty`
`1209`		`- * buffers. But at shutdowntime, we write all dirty buffers.`
	`1209`	`+ * buffers. But at shutdownor end of recovery, we write all dirty buffers.`
`1210`	`1210`	`*/`
`1211`		`-if (!(flags&CHECKPOINT_IS_SHUTDOWN))`
	`1211`	`+if (!((flags&CHECKPOINT_IS_SHUTDOWN)\|\| (flags&CHECKPOINT_END_OF_RECOVERY)))`
`1212`	`1212`	`mask \|=BM_PERMANENT;`
`1213`	`1213`
`1214`	`1214`	`/*`

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!

Commit64e196b

File tree

1 file changed

1 file changed

`‎src/backend/storage/buffer/bufmgr.c`

0 commit comments