NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commitf588548

committed

Fix order of arguments to SubTransSetParent().

ProcessTwoPhaseBuffer (formerly StandbyRecoverPreparedTransactions)mixed up the parent and child XIDs when calling SubTransSetParent torecord the transactions' relationship in pg_subtrans.Remarkably, analysis by Simon Riggs suggests that this doesn't lead tovisible problems (at least, not in non-Assert builds). That mightexplain why we'd not noticed it before. Nonetheless, it's surely wrong.This code was born broken, so back-patch to all supported branches.Discussion:https://postgr.es/m/20110.1492905318@sss.pgh.pa.us

1 parent11927e5 commitf588548Copy full SHA for f588548

File tree

1 file changed

-1

lines changed

src/backend/access/transam
- twophase.c

1 file changed

-1

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1887,7 +1887,7 @@ StandbyRecoverPreparedTransactions(bool overwriteOK)`
`1887`	`1887`	`TransactionIdsubxid=subxids[i];`
`1888`	`1888`
`1889`	`1889`	`Assert(TransactionIdFollows(subxid,xid));`
`1890`		`-SubTransSetParent(xid,subxid,overwriteOK);`
	`1890`	`+SubTransSetParent(subxid,xid,overwriteOK);`
`1891`	`1891`	`}`
`1892`	`1892`
`1893`	`1893`	`pfree(buf);`

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!

Commitf588548

File tree

1 file changed

1 file changed

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

0 commit comments