bzr commit into mysql-6.0-falcon branch (cpowers:2792) Bug#38041

[Christopher Powers]

#At file:///home/cpowers/work/dev/dev-05/mysql/

 2792 Christopher Powers	2008-08-20
      Bug #38041 "Bizarre errors when ALTER ADD/DROP KEY on Falcon tables"
      - Corrected mixup with StorageTableShare::syncObject and ::syncIndexes
      
      
      Bug #38377 "Options falcon_serial_log_dir does not have any effect"
      - Temporarily disabled fatal exception in Configuration::Configuration() until
ScanDir.isDirectory() be fixed on Linux.
modified:
  storage/falcon/Configuration.cpp
  storage/falcon/StorageTableShare.cpp

per-file messages:
  storage/falcon/Configuration.cpp
    Check of falcon_serial_log_dir in Configuration::Configuration() still throws
    an exception even when the directory is valid. Disabled exception until this is fixed.
  storage/falcon/StorageTableShare.cpp
    StorageTableShare::lock/unlock() now locks syncObject instead of syncIndexes
=== modified file 'storage/falcon/Configuration.cpp'
--- a/storage/falcon/Configuration.cpp	2008-08-19 14:27:42 +0000
+++ b/storage/falcon/Configuration.cpp	2008-08-20 19:40:10 +0000
@@ -158,7 +158,7 @@ Configuration::Configuration(const char 
 					"the user before initializing Falcon.\n", 
 					falcon_serial_log_dir
 				);
-			throw SQLEXCEPTION (FILE_ACCESS_ERROR, "Invalid serial log directory path \"%s\"",
falcon_serial_log_dir);
+//			throw SQLEXCEPTION (FILE_ACCESS_ERROR, "Invalid serial log directory path \"%s\"",
falcon_serial_log_dir);
 			}
 		}
 #else

=== modified file 'storage/falcon/StorageTableShare.cpp'
--- a/storage/falcon/StorageTableShare.cpp	2008-08-19 03:33:01 +0000
+++ b/storage/falcon/StorageTableShare.cpp	2008-08-20 19:40:10 +0000
@@ -94,14 +94,12 @@ StorageTableShare::~StorageTableShare(vo
 
 void StorageTableShare::lock(bool exclusiveLock)
 {
-	//syncObject->lock(NULL, (exclusiveLock) ? Exclusive : Shared);
-	syncIndexes->lock(NULL, (exclusiveLock) ? Exclusive : Shared);
+	syncObject->lock(NULL, (exclusiveLock) ? Exclusive : Shared);
 }
 
 void StorageTableShare::unlock(void)
 {
-	//syncObject->unlock();
-	syncIndexes->unlock();
+	syncObject->unlock();
 }
 
 void StorageTableShare::lockIndexes(bool exclusiveLock)