NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit66fbcb0

committed

Avoid hot standby cancels from VAC FREEZE

VACUUM FREEZE generated false cancelations of standby queries on anotherwise idle master. Caused by an off-by-one error on cutoff_xidwhich goes back to original commit.Backpatch to all versions 9.0+Analysis and report by Marco NenciariniBug fix by Simon Riggs

1 parent7d60b2a commit66fbcb0Copy full SHA for 66fbcb0

File tree

1 file changed

-1

lines changed

src/backend/access/heap
- heapam.c

1 file changed

-1

lines changed

`‎src/backend/access/heap/heapam.c`

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -7572,9 +7572,12 @@ heap_xlog_freeze_page(XLogReaderState *record)`
`7572`	`7572`	`if (InHotStandby)`
`7573`	`7573`	`{`
`7574`	`7574`	`RelFileNodernode;`
	`7575`	`+TransactionIdlatestRemovedXid=cutoff_xid;`
	`7576`	`+`
	`7577`	`+TransactionIdRetreat(latestRemovedXid);`
`7575`	`7578`
`7576`	`7579`	`XLogRecGetBlockTag(record,0,&rnode,NULL,NULL);`
`7577`		`-ResolveRecoveryConflictWithSnapshot(cutoff_xid,rnode);`
	`7580`	`+ResolveRecoveryConflictWithSnapshot(latestRemovedXid,rnode);`
`7578`	`7581`	`}`
`7579`	`7582`
`7580`	`7583`	`if (XLogReadBufferForRedo(record,0,&buffer)==BLK_NEEDS_REDO)`

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!

Commit66fbcb0

File tree

1 file changed

1 file changed

`‎src/backend/access/heap/heapam.c`

0 commit comments