NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commit0874d4f

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 parent33f3bbc commit0874d4fCopy full SHA for 0874d4f

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
`@@ -2142,7 +2142,7 @@ ProcessTwoPhaseBuffer(TransactionId xid,`
`2142`	`2142`	`}`
`2143`	`2143`
`2144`	`2144`	`if (setParent)`
`2145`		`-SubTransSetParent(xid,subxid,overwriteOK);`
	`2145`	`+SubTransSetParent(subxid,xid,overwriteOK);`
`2146`	`2146`	`}`
`2147`	`2147`
`2148`	`2148`	`returnbuf;`

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!

Commit0874d4f

File tree

1 file changed

1 file changed

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

0 commit comments