diff options
| author | Simon Riggs <simon@2ndQuadrant.com> | 2015-08-14 14:19:28 +0100 |
|---|---|---|
| committer | Simon Riggs <simon@2ndQuadrant.com> | 2015-08-14 14:19:28 +0100 |
| commit | 47167b7907a802ed39b179c8780b76359468f076 (patch) | |
| tree | 0f52c9314b69cd80d7a4284f8d405db84122c4b9 /src/backend/commands/tablecmds.c | |
| parent | f16d52269a196f7f303abe3b978d95ade265f05f (diff) | |
Reduce lock levels for ALTER TABLE SET autovacuum storage options
Reduce lock levels down to ShareUpdateExclusiveLock for all autovacuum-related
relation options when setting them using ALTER TABLE.
Add infrastructure to allow varying lock levels for relation options in later
patches. Setting multiple options together uses the highest lock level required
for any option. Works for both main and toast tables.
FabrÃzio Mello, reviewed by Michael Paquier, mild edit and additional regression
tests from myself
Diffstat (limited to 'src/backend/commands/tablecmds.c')
| -rw-r--r-- | src/backend/commands/tablecmds.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c index 970abd4b50c..126b11923f0 100644 --- a/src/backend/commands/tablecmds.c +++ b/src/backend/commands/tablecmds.c @@ -3038,16 +3038,12 @@ AlterTableGetLockLevel(List *cmds) * are set here for tables, views and indexes; for historical * reasons these can all be used with ALTER TABLE, so we can't * decide between them using the basic grammar. - * - * XXX Look in detail at each option to determine lock level, - * e.g. cmd_lockmode = GetRelOptionsLockLevel((List *) - * cmd->def); */ case AT_SetRelOptions: /* Uses MVCC in getIndexes() and * getTables() */ case AT_ResetRelOptions: /* Uses MVCC in getIndexes() and * getTables() */ - cmd_lockmode = AccessExclusiveLock; + cmd_lockmode = AlterTableGetRelOptionsLockLevel((List *) cmd->def); break; default: /* oops */ |