NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitb5f6341

committed

Fix one-off bug causing missing commit timestamps for subtransactions

The logic in charge of writing commit timestamps (enabled withtrack_commit_timestamp) for subtransactions had a one-bug bug,where it would be possible that commit timestamps go missing for thelast subtransaction committed.While on it, simplify a bit the iteration logic in the loop writing thecommit timestamps, as per suggestions from Kyotaro Horiguchi and TomLane, so as some variable initializations are not part of the loopitself.Issue introduced in73c986a.Analyzed-by: Alex KingsboroughAuthor: Alex Kingsborough, Kyotaro HoriguchiDiscussion:https://postgr.es/m/73A66172-4050-4F2A-B7F1-13508EDA2144@amazon.comBackpatch-through: 10

1 parent4828a80 commitb5f6341Copy full SHA for b5f6341

File tree

1 file changed

-3

lines changed

src/backend/access/transam
- commit_ts.c

1 file changed

-3

lines changed

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

Lines changed: 5 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,9 @@ TransactionTreeSetCommitTsData(TransactionId xid, int nsubxids,`
`184`	`184`	`* subxid not on the previous page as head. This way, we only have to`
`185`	`185`	`* lock/modify each SLRU page once.`
`186`	`186`	`*/`
`187`		`-for (i=0,headxid=xid;;)`
	`187`	`+headxid=xid;`
	`188`	`+i=0;`
	`189`	`+for (;;)`
`188`	`190`	`{`
`189`	`191`	`intpageno=TransactionIdToCTsPage(headxid);`
`190`	`192`	`intj;`
`@@ -200,15 +202,15 @@ TransactionTreeSetCommitTsData(TransactionId xid, int nsubxids,`
`200`	`202`	`pageno);`
`201`	`203`
`202`	`204`	`/* if we wrote out all subxids, we're done. */`
`203`		`-if (j+1>=nsubxids)`
	`205`	`+if (j >=nsubxids)`
`204`	`206`	`break;`
`205`	`207`
`206`	`208`	`/*`
`207`	`209`	`* Set the new head and skip over it, as well as over the subxids we`
`208`	`210`	`* just wrote.`
`209`	`211`	`*/`
`210`	`212`	`headxid=subxids[j];`
`211`		`-i+=j-i+1;`
	`213`	`+i=j+1;`
`212`	`214`	`}`
`213`	`215`
`214`	`216`	`/* update the cached value in shared memory */`

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!

Commitb5f6341

File tree

1 file changed

1 file changed

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

0 commit comments