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

Use list_length() != 1 in SingleReplicatedTable(). 

ShardPlacementList's implementation can return NIL. In previous implementation
we got a segmentation fault in this case. The relation can be dropped after
getting distributed table list but before calling SingleReplicatedTable().
pull/1751/head
Hadi Moshayedi 2017-10-31 21:38:20 -04:00 committed by Hadi Moshayedi
parent 7691991cb5
commit 7280774cf4
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/backend/distributed/utils/shardinterval_utils.c
View File

@ -390,7 +390,7 @@ SingleReplicatedTable(Oid relationId)
uint64 shardId = (*shardIdPointer);
List *shardPlacementList = ShardPlacementList(shardId);
if (shardPlacementList->length > 1)
if (list_length(shardPlacementList) != 1)
{
return false;
}