fix incorrect condition

funbringer · funbringer · commit 44eb6ceafb75 · 2016-08-30T11:17:15.000+03:00
diff --git a/README.md b/README.md
@@ -29,16 +29,14 @@ WHERE id = 150
 Based on the partitioning type and condition's operator, `pg_pathman` searches for the corresponding partitions and builds the plan. Currently `pg_pathman` supports two partitioning schemes:
 
 * **RANGE** - maps rows to partitions using partitioning key ranges assigned to each partition. Optimization is achieved by using the binary search algorithm;
-* **HASH** - maps rows to partitions using a generic hash function (only *integer* attributes are supported at the moment).
+* **HASH** - maps rows to partitions using a generic hash function.
 
 More interesting features are yet to come. Stay tuned!
 
 ## Roadmap
- * Replace INSERT triggers with a custom node (aka **PartitionFilter**)
- * Implement [concurrent partitioning](https://github.com/postgrespro/pg_pathman/tree/concurrent_part) (much more responsive)
- * Implement HASH partitioning for non-integer attributes
- * Optimize hash join (both tables are partitioned by join key)
- * Implement LIST partitioning scheme
+
+ * Implement LIST partitioning scheme;
+ * Optimize hash join (both tables are partitioned by join key).
 
 ## Installation guide
 To install `pg_pathman`, execute this in the module's directory:
diff --git a/src/pathman_workers.c b/src/pathman_workers.c
@@ -615,7 +615,8 @@ partition_table_concurrently(PG_FUNCTION_ARGS)
 
 		/* Oops, looks like we already have BGWorker for this table */
 		if (cur_slot->relid == relid &&
-			cur_slot->dbid == MyDatabaseId)
+			cur_slot->dbid == MyDatabaseId &&
+			cur_slot->worker_status != CPS_FREE)
 		{
 			/* Unlock current slot */
 			SpinLockRelease(&cur_slot->mutex);

Original file line number	Diff line number	Diff line change
`@@ -615,7 +615,8 @@ partition_table_concurrently(PG_FUNCTION_ARGS)`
`615`	`615`
`616`	`616`	`/* Oops, looks like we already have BGWorker for this table */`
`617`	`617`	`if (cur_slot->relid == relid &&`
`618`		`- cur_slot->dbid == MyDatabaseId)`
	`618`	`+ cur_slot->dbid == MyDatabaseId &&`
	`619`	`+ cur_slot->worker_status != CPS_FREE)`
`619`	`620`	`{`
`620`	`621`	`/* Unlock current slot */`
`621`	`622`	`SpinLockRelease(&cur_slot->mutex);`