farrokhi/pg_repack
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

acquire AccessShareLock on parent and children in repack_get_table_and_inheritors()

Browse Source
This commit is contained in:
Dmitry Ivanov 2017-04-23 21:22:54 +03:00
parent 4df065891c
commit 0901b82588
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/repack.c
View File

@ -32,6 +32,7 @@
#include "commands/tablecmds.h" #include "commands/tablecmds.h"
#include "commands/trigger.h" #include "commands/trigger.h"
#include "miscadmin.h" #include "miscadmin.h"
#include "storage/lmgr.h"
#include "utils/array.h" #include "utils/array.h"
#include "utils/builtins.h" #include "utils/builtins.h"
#include "utils/lsyscache.h" #include "utils/lsyscache.h"
@ -1314,7 +1315,14 @@ repack_get_table_and_inheritors(PG_FUNCTION_ARGS)
ListCell *lc; ListCell *lc;
int i; int i;
relations = find_all_inheritors(parent, NoLock, NULL); LockRelationOid(parent, AccessShareLock);
/* Check that parent table exists */
if (!SearchSysCacheExists1(RELOID, ObjectIdGetDatum(parent)))
PG_RETURN_ARRAYTYPE_P(construct_empty_array(OIDOID));
/* Also check that children exist */
relations = find_all_inheritors(parent, AccessShareLock, NULL);
relations_array_size = list_length(relations); relations_array_size = list_length(relations);
if (relations_array_size == 0) if (relations_array_size == 0)