mirror of https://github.com/citusdata/citus.git
Move identity column existance check to ConvertTable()
parent
55ebabc875
commit
4fb37f33cc
|
@ -406,8 +406,6 @@ UndistributeTable(TableConversionParameters *params)
|
||||||
params->shardCountIsNull = true;
|
params->shardCountIsNull = true;
|
||||||
TableConversionState *con = CreateTableConversion(params);
|
TableConversionState *con = CreateTableConversion(params);
|
||||||
|
|
||||||
ErrorIfTableHasIdentityColumn(params->relationId);
|
|
||||||
|
|
||||||
return ConvertTable(con);
|
return ConvertTable(con);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -437,7 +435,6 @@ AlterDistributedTable(TableConversionParameters *params)
|
||||||
EnsureHashDistributedTable(params->relationId);
|
EnsureHashDistributedTable(params->relationId);
|
||||||
|
|
||||||
ErrorIfUnsupportedCascadeObjects(params->relationId);
|
ErrorIfUnsupportedCascadeObjects(params->relationId);
|
||||||
ErrorIfTableHasIdentityColumn(params->relationId);
|
|
||||||
|
|
||||||
params->conversionType = ALTER_DISTRIBUTED_TABLE;
|
params->conversionType = ALTER_DISTRIBUTED_TABLE;
|
||||||
TableConversionState *con = CreateTableConversion(params);
|
TableConversionState *con = CreateTableConversion(params);
|
||||||
|
@ -502,7 +499,6 @@ AlterTableSetAccessMethod(TableConversionParameters *params)
|
||||||
}
|
}
|
||||||
|
|
||||||
ErrorIfUnsupportedCascadeObjects(params->relationId);
|
ErrorIfUnsupportedCascadeObjects(params->relationId);
|
||||||
ErrorIfTableHasIdentityColumn(params->relationId);
|
|
||||||
|
|
||||||
params->conversionType = ALTER_TABLE_SET_ACCESS_METHOD;
|
params->conversionType = ALTER_TABLE_SET_ACCESS_METHOD;
|
||||||
params->shardCountIsNull = true;
|
params->shardCountIsNull = true;
|
||||||
|
@ -932,6 +928,7 @@ CopyTableConversionReturnIntoCurrentContext(TableConversionReturn *tableConversi
|
||||||
static TableConversionReturn *
|
static TableConversionReturn *
|
||||||
ConvertTable(TableConversionState *con)
|
ConvertTable(TableConversionState *con)
|
||||||
{
|
{
|
||||||
|
ErrorIfTableHasIdentityColumn(con->relationId);
|
||||||
/*
|
/*
|
||||||
* when there are many partitions or colocated tables, memory usage is
|
* when there are many partitions or colocated tables, memory usage is
|
||||||
* accumulated. Free context for each call to ConvertTable.
|
* accumulated. Free context for each call to ConvertTable.
|
||||||
|
|
Loading…
Reference in New Issue