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]
