diff options
| author | Noah Misch <noah@leadboat.com> | 2024-06-27 19:21:05 -0700 |
|---|---|---|
| committer | Noah Misch <noah@leadboat.com> | 2024-06-27 19:21:13 -0700 |
| commit | 7af775bbae5724272f3b72766df977f07fa5455a (patch) | |
| tree | ac1c73172294f68619b0340930e785f389383f16 | |
| parent | e2eefb97e31a64b48bb22fe53d0101586891d01f (diff) | |
| download | postgresql-7af775bbae5724272f3b72766df977f07fa5455a.tar.gz postgresql-7af775bbae5724272f3b72766df977f07fa5455a.zip | |
AccessExclusiveLock new relations just after assigning the OID.
This has no user-visible, important consequences, since other sessions'
catalog scans can't find the relation until we commit. However, this
unblocks introducing a rule about locks required to heap_update() a
pg_class row. CREATE TABLE has been acquiring this lock eventually, but
it can heap_update() pg_class.relchecks earlier. create_toast_table()
has been acquiring only ShareLock. Back-patch to v12 (all supported
versions), the plan for the commit relying on the new rule.
Reviewed (in an earlier version) by Robert Haas.
Discussion: https://postgr.es/m/20240611024525.9f.nmisch@google.com
| -rw-r--r-- | src/backend/catalog/heap.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/backend/catalog/heap.c b/src/backend/catalog/heap.c index a98a5115837..8cf25845222 100644 --- a/src/backend/catalog/heap.c +++ b/src/backend/catalog/heap.c @@ -1227,6 +1227,13 @@ heap_create_with_catalog(const char *relname, } /* + * Other sessions' catalog scans can't find this until we commit. Hence, + * it doesn't hurt to hold AccessExclusiveLock. Do it here so callers + * can't accidentally vary in their lock mode or acquisition timing. + */ + LockRelationOid(relid, AccessExclusiveLock); + + /* * Determine the relation's initial permissions. */ if (use_user_acl) |