NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit53336e8

committed

Release lock after encountering bogs row in vac_truncate_clog()

When vac_truncate_clog() encounters bogus datfrozenxid / datminmxid values, itreturns early. Unfortunately, until now, it did not releaseWrapLimitsVacuumLock. If the backend later tries to acquireWrapLimitsVacuumLock, the session / autovacuum worker hangs in anuncancellable way. Similarly, other sessions will hang waiting for thelock. However, if the backend holding the lock exited or errored out for somereason, the lock was released.The bug was introduced as a side effect of566372b.It is interesting that there are no production reports of this problem. Thatis likely due to a mix of bugs leading to bogus values having gotten lesscommon, process exit releasing locks and instances of hangs being hard todebug for "normal" users.Discussion:https://postgr.es/m/20230621221208.vhsqgduwfpzwxnpg@awork3.anarazel.de

1 parent7fffcc2 commit53336e8Copy full SHA for 53336e8

File tree

1 file changed

-0

lines changed

src/backend/commands
- vacuum.c

1 file changed

-0

lines changed

`‎src/backend/commands/vacuum.c`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1639,12 +1639,16 @@ vac_truncate_clog(TransactionId frozenXID,`
`1639`	`1639`	`ereport(WARNING,`
`1640`	`1640`	`(errmsg("some databases have not been vacuumed in over 2 billion transactions"),`
`1641`	`1641`	`errdetail("You might have already suffered transaction-wraparound data loss.")));`
	`1642`	`+LWLockRelease(WrapLimitsVacuumLock);`
`1642`	`1643`	`return;`
`1643`	`1644`	`}`
`1644`	`1645`
`1645`	`1646`	`/* chicken out if data is bogus in any other way */`
`1646`	`1647`	`if (bogus)`
	`1648`	`+{`
	`1649`	`+LWLockRelease(WrapLimitsVacuumLock);`
`1647`	`1650`	`return;`
	`1651`	`+}`
`1648`	`1652`
`1649`	`1653`	`/*`
`1650`	`1654`	`* Advance the oldest value for commit timestamps before truncating, so`

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!

Commit53336e8

File tree

1 file changed

1 file changed

`‎src/backend/commands/vacuum.c`

0 commit comments