Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitdbf3f97

Browse files
committed
Warn if LOCKTAG_TUPLE is held at commit, under debug_assertions.
The current use always releases this locktag. A planned use willcontinue that intent. It will involve more areas of code, making unlockomissions easier. Warn under debug_assertions, like we do for variousresource leaks. Back-patch to v12 (all supported versions), the planfor the commit of the new use.Reviewed by Heikki Linnakangas.Discussion:https://postgr.es/m/20240512232923.aa.nmisch@google.com
1 parentac30021 commitdbf3f97

File tree

1 file changed

+10
-0
lines changed
  • src/backend/storage/lmgr

1 file changed

+10
-0
lines changed

‎src/backend/storage/lmgr/lock.c

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2305,6 +2305,16 @@ LockReleaseAll(LOCKMETHODID lockmethodid, bool allLocks)
23052305
locallock->numLockOwners=0;
23062306
}
23072307

2308+
#ifdefUSE_ASSERT_CHECKING
2309+
2310+
/*
2311+
* Tuple locks are currently held only for short durations within a
2312+
* transaction. Check that we didn't forget to release one.
2313+
*/
2314+
if (LOCALLOCK_LOCKTAG(*locallock)==LOCKTAG_TUPLE&& !allLocks)
2315+
elog(WARNING,"tuple lock held at commit");
2316+
#endif
2317+
23082318
/*
23092319
* If the lock or proclock pointers are NULL, this lock was taken via
23102320
* the relation fast-path (and is not known to have been transferred).

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp