mark 'prel' invalid before locking parent

funbringer · funbringer · commit 4bd4295b1403 · 2016-10-24T14:10:28.000+03:00
diff --git a/src/relation_info.c b/src/relation_info.c
@@ -104,6 +104,9 @@ refresh_pathman_relation_info(Oid relid,
 		FreeRangesArray(prel);
 	}
 
+	/* First we assume that this entry is invalid */
+	prel->valid		= false;
+
 	/* Try locking parent, exit fast if 'allow_incomplete' */
 	if (allow_incomplete)
 	{
@@ -121,9 +124,6 @@ refresh_pathman_relation_info(Oid relid,
 		return NULL; /* exit */
 	}
 
-	/* First we assume that this entry is invalid */
-	prel->valid		= false;
-
 	/* Make both arrays point to NULL */
 	prel->children	= NULL;
 	prel->ranges	= NULL;

Original file line number	Diff line number	Diff line change
`@@ -104,6 +104,9 @@ refresh_pathman_relation_info(Oid relid,`
`104`	`104`	`FreeRangesArray(prel);`
`105`	`105`	`}`
`106`	`106`
	`107`	`+ /* First we assume that this entry is invalid */`
	`108`	`+ prel->valid = false;`
	`109`	`+`
`107`	`110`	`/* Try locking parent, exit fast if 'allow_incomplete' */`
`108`	`111`	`if (allow_incomplete)`
`109`	`112`	`{`
`@@ -121,9 +124,6 @@ refresh_pathman_relation_info(Oid relid,`
`121`	`124`	`return NULL; /* exit */`
`122`	`125`	`}`
`123`	`126`
`124`		`- /* First we assume that this entry is invalid */`
`125`		`- prel->valid = false;`
`126`		`-`
`127`	`127`	`/* Make both arrays point to NULL */`
`128`	`128`	`prel->children = NULL;`
`129`	`129`	`prel->ranges = NULL;`