NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit053ad56

committed

Fix lock level used for partition when detaching it

For probably bogus reasons, we acquire only AccessShareLock on thepartition when we try to detach it from its parent partitioned table.This can cause ugly things to happen if another transaction is doingany sort of DDL to the partition concurrently.Upgrade that lock to ShareUpdateExclusiveLock, which per discussionseems to be the minimum needed.Reported by Robert Haas.Discussion:https://postgr.es/m/CA+TgmoYruJQ+2qnFLtF1xQtr71pdwgfxy3Ziy-TxV28M6pEmyA@mail.gmail.com

1 parent4e4c0d6 commit053ad56Copy full SHA for 053ad56

File tree

1 file changed

-1

lines changed

src/backend/commands
- tablecmds.c

1 file changed

-1

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -14726,7 +14726,7 @@ ATExecDetachPartition(Relation rel, RangeVar *name)`
`14726`	`14726`	`if (OidIsValid(defaultPartOid))`
`14727`	`14727`	`LockRelationOid(defaultPartOid,AccessExclusiveLock);`
`14728`	`14728`
`14729`		`-partRel=heap_openrv(name,AccessShareLock);`
	`14729`	`+partRel=heap_openrv(name,ShareUpdateExclusiveLock);`
`14730`	`14730`
`14731`	`14731`	`/* All inheritance related checks are performed within the function */`
`14732`	`14732`	`RemoveInheritance(partRel,rel);`

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!

Commit053ad56

File tree

1 file changed

1 file changed

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

0 commit comments