提交 875724ba 编写于 作者: A Ashwin Agrawal

Resolve GPDB_91_MERGE_FIXME related to locks in tablecmds.c

The lock level looks fine hence resolving the fixme's.
上级 281cfd69
......@@ -3678,10 +3678,6 @@ AlterTableGetLockLevel(List *cmds)
case AT_PartSplit:
case AT_PartTruncate:
case AT_PartAddInternal:
/*
* GPDB_91_MERGE_FIXME: Is AccessExclusiveLock unnecessarily strict for
* some of these?
*/
cmd_lockmode = AccessExclusiveLock;
break;
......@@ -14204,7 +14200,6 @@ ATPExecPartAlter(List **wqueue, AlteredTableInfo *tab, Relation *rel,
}
/* execute the command */
/* GPDB_91_MERGE_FIXME: is this lock mode right? */
ATExecCmd(wqueue, tab, (rel2 ? &rel2 : rel), atc, AccessExclusiveLock);
if (!bPartitionCmd)
......@@ -14500,7 +14495,7 @@ exchange_part_inheritance(Oid oldrelid, Oid newrelid)
ATExecDropInherit(oldrel,
makeRangeVar(get_namespace_name(parent->rd_rel->relnamespace),
RelationGetRelationName(parent), -1),
AccessExclusiveLock, /* GPDB_91_MERGE_FIXME: lock mode? */
AccessExclusiveLock, /* Not used for anything in ATExecDropInherit() */
true);
inherit_parent(parent, newrel, true /* it's a partition */, NIL);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册