external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Enable partitioned citus local tables

pull/5179/head
Ahmet Gedemenli 2021-08-16 15:55:48 +03:00
parent 6e8b19984e
commit c4e2fe3500
2 changed files with 14 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/backend/distributed/commands/citus_add_local_table_to_metadata.c
View File

@ -330,6 +330,17 @@ CreateCitusLocalTable(Oid relationId, bool cascadeViaForeignKeys)
attnumList); attnumList);
FinalizeCitusLocalTableCreation(shellRelationId, dependentSequenceList); FinalizeCitusLocalTableCreation(shellRelationId, dependentSequenceList);
/* if this table is partitioned table, add its partitions to metadata too */
if (PartitionedTable(relationId))
{
List *partitionList = PartitionList(relationId);
Oid partitionRelationId = InvalidOid;
foreach_oid(partitionRelationId, partitionList)
{
CreateCitusLocalTable(partitionRelationId, false);
}
}
} }
@ -387,20 +398,14 @@ ErrorIfUnsupportedCitusLocalTableKind(Oid relationId)
"relationships", relationName))); "relationships", relationName)));
} }
if (PartitionTable(relationId))
{
ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
errmsg("cannot add local table \"%s\" to metadata, local tables "
"added to metadata cannot be partition of other tables ",
relationName)));
}
char relationKind = get_rel_relkind(relationId); char relationKind = get_rel_relkind(relationId);
if (!(relationKind == RELKIND_RELATION || relationKind == RELKIND_FOREIGN_TABLE)) if (!(relationKind == RELKIND_RELATION || relationKind == RELKIND_FOREIGN_TABLE || relationKind == RELKIND_PARTITIONED_TABLE))
{ {
ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
errmsg("cannot add local table \"%s\" to metadata, only regular " errmsg("cannot add local table \"%s\" to metadata, only regular "
"tables and foreign tables can be added to citus metadata ", "tables, foreign tables and partitioned tables can be "
"added to citus metadata ",
relationName))); relationName)));
} }

6
src/backend/distributed/transaction/relation_access_tracking.c
View File

@ -227,12 +227,6 @@ PlacementAccessTypeToText(ShardPlacementAccessType accessType)
static void static void
RecordRelationAccessBase(Oid relationId, ShardPlacementAccessType accessType) RecordRelationAccessBase(Oid relationId, ShardPlacementAccessType accessType)
{ {
/*
* We call this only for reference tables, and we don't support partitioned
* reference tables.
*/
Assert(!PartitionedTable(relationId) && !PartitionTable(relationId));
/* make sure that this is not a conflicting access */ /* make sure that this is not a conflicting access */
CheckConflictingRelationAccesses(relationId, accessType); CheckConflictingRelationAccesses(relationId, accessType);