NotificationsYou must be signed in to change notification settings
Fork28
Star153

Commitb277057

committed

Fix broken Assert() introduced by 8e9a16ab8f7f0e58

Don't assert MultiXactIdIsRunning if the multi came from a tuple thathad been share-locked and later copied over to the new cluster bypg_upgrade. Doing that causes an error to be raised unnecessarily:MultiXactIdIsRunning is not open to the possibility that its argumentcame from a pg_upgraded tuple, and all its other callers are alreadychecking; but such multis cannot, obviously, have transactions stillrunning, so the assert is pointless.Noticed while investigating the bogus pg_multixact/offsets/0000 fileleft over by pg_upgrade, as reported by Andres Freund inhttp://www.postgresql.org/message-id/20140530121631.GE25431@alap3.anarazel.deBackpatch to 9.3, as the commit that introduced the buglet.

1 parent1147035 commitb277057Copy full SHA for b277057

File tree

1 file changed

-1

lines changed

src/backend/access/heap
- heapam.c

1 file changed

-1

lines changed

`‎src/backend/access/heap/heapam.c‎`

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -5518,8 +5518,14 @@ FreezeMultiXactId(MultiXactId multi, uint16 t_infomask,`
`5518`	`5518`	`* was a locker only, it can be removed without any further`
`5519`	`5519`	`* consideration; but if it contained an update, we might need to`
`5520`	`5520`	`* preserve it.`
	`5521`	`+ *`
	`5522`	`+ * Don't assert MultiXactIdIsRunning if the multi came from a`
	`5523`	`+ * pg_upgrade'd share-locked tuple, though, as doing that causes an`
	`5524`	`+ * error to be raised unnecessarily.`
`5521`	`5525`	`*/`
`5522`		`-Assert(!MultiXactIdIsRunning(multi));`
	`5526`	`+Assert((!(t_infomask&HEAP_LOCK_MASK)&&`
	`5527`	`+HEAP_XMAX_IS_LOCKED_ONLY(t_infomask))\|\|`
	`5528`	`+ !MultiXactIdIsRunning(multi));`
`5523`	`5529`	`if (HEAP_XMAX_IS_LOCKED_ONLY(t_infomask))`
`5524`	`5530`	`{`
`5525`	`5531`	`*flags \|=FRM_INVALIDATE_XMAX;`

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!

Commitb277057

File tree

1 file changed

1 file changed

`‎src/backend/access/heap/heapam.c‎`

0 commit comments