NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitef41c3f

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 parent04875ae commitef41c3fCopy full SHA for ef41c3f

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
`@@ -465,7 +465,7 @@ XLogReadRecord(XLogReaderState state, char *errormsg)`
`465`	`465`	`*/`
`466`	`466`	`if (pageHeader->xlp_info&XLP_FIRST_IS_OVERWRITE_CONTRECORD)`
`467`	`467`	`{`
`468`		`-state->overwrittenRecPtr=state->currRecPtr;`
	`468`	`+state->overwrittenRecPtr=RecPtr;`
`469`	`469`	`ResetDecoder(state);`
`470`	`470`	`RecPtr=targetPagePtr;`
`471`	`471`	`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!

Commitef41c3f

File tree

1 file changed

1 file changed

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

0 commit comments