NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit9c6ab1e

committed

Fix determination of broken LSN in OVERWRITTEN_CONTRECORD

In commitff9f111 I mixed up inconsistent definitions of the LSN ofthe first record in a page, when the previous record ends exactly at thepage boundary. The correct LSN is adjusted to skip the WAL page header;I failed to use that when setting XLogReaderState->overwrittenRecPtr,so at WAL replay time VerifyOverwriteContrecord would refuse to letreplay continue past that record.Backpatch to 10. 9.6 also contains this bug, but it's no longer beingmaintained.Discussion:https://postgr.es/m/45597.1637694259@sss.pgh.pa.us

1 parent0d335cb commit9c6ab1eCopy full SHA for 9c6ab1e

File tree

1 file changed

-1

lines changed

src/backend/access/transam
- xlogreader.c

1 file changed

-1

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -415,7 +415,7 @@ XLogReadRecord(XLogReaderState state, XLogRecPtr RecPtr, char *errormsg)`
`415`	`415`	`*/`
`416`	`416`	`if (pageHeader->xlp_info&XLP_FIRST_IS_OVERWRITE_CONTRECORD)`
`417`	`417`	`{`
`418`		`-state->overwrittenRecPtr=state->currRecPtr;`
	`418`	`+state->overwrittenRecPtr=RecPtr;`
`419`	`419`	`ResetDecoder(state);`
`420`	`420`	`RecPtr=targetPagePtr;`
`421`	`421`	`gotorestart;`

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!

Commit9c6ab1e

File tree

1 file changed

1 file changed

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

0 commit comments