AFAICS, we now longer use a global variable to record the fact that
we're in the midst of building an index. Therefore, this patch removes
some comments (in access/hash/hash.c and access/nbtree/nbtree.c) that
claim we do.

I'll apply this to CVS tomorrow barring any objections.

-Neil

--- src/backend/access/hash/hash.c
+++ src/backend/access/hash/hash.c
@@ -44,11 +44,6 @@
 
 /*
  *	hashbuild() -- build a new hash index.
- *
- *		We use a global variable to record the fact that we're creating
- *		a new index.  This is used to avoid high-concurrency locking,
- *		since the index won't be visible until this transaction commits
- *		and since building is guaranteed to be single-threaded.
  */
 Datum
 hashbuild(PG_FUNCTION_ARGS)
--- src/backend/access/nbtree/nbtree.c
+++ src/backend/access/nbtree/nbtree.c
@@ -60,11 +60,6 @@
 
 /*
  *	btbuild() -- build a new btree index.
- *
- *		We use a global variable to record the fact that we're creating
- *		a new index.  This is used to avoid high-concurrency locking,
- *		since the index won't be visible until this transaction commits
- *		and since building is guaranteed to be single-threaded.
  */
 Datum
 btbuild(PG_FUNCTION_ARGS)
---------------------------(end of broadcast)---------------------------
TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]

Reply via email to