NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit1c14261

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 parent164d439 commit1c14261Copy full SHA for 1c14261

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
`@@ -13865,7 +13865,7 @@ ATExecDetachPartition(Relation rel, RangeVar *name)`
`13865`	`13865`	`new_repl[Natts_pg_class];`
`13866`	`13866`	`ObjectAddressaddress;`
`13867`	`13867`
`13868`		`-partRel=heap_openrv(name,AccessShareLock);`
	`13868`	`+partRel=heap_openrv(name,ShareUpdateExclusiveLock);`
`13869`	`13869`
`13870`	`13870`	`/* All inheritance related checks are performed within the function */`
`13871`	`13871`	`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!

Commit1c14261

File tree

1 file changed

1 file changed

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

0 commit comments