Connector/NET commit: r809 - in trunk: Driver/Source TestSuite/Source

[rburnett]

Modified:
   trunk/Driver/Source/MySqlPool.cs
   trunk/TestSuite/Source/PoolingTests.cs
Log:
Fixed problem where an attempt to open a connection max pool size times while the server
is down will prevent any further attempts due to the pool semaphore being full. (Bug
#29409)  


Modified: trunk/Driver/Source/MySqlPool.cs
===================================================================
--- trunk/Driver/Source/MySqlPool.cs	2007-07-24 00:13:06 UTC (rev 808)
+++ trunk/Driver/Source/MySqlPool.cs	2007-07-24 00:13:44 UTC (rev 809)
@@ -135,7 +135,8 @@
                 // if we don't have an idle connection but we have room for a new
                 // one, then create it here.
                 if (!HasIdleConnections)
-                    CreateNewPooledConnection();
+                    if (!CreateNewPooledConnection())
+                        return null;
 
                 Driver d = CheckoutConnection();
                 if (d != null)
@@ -146,11 +147,20 @@
         /// <summary>
         /// It is assumed that this method is only called from inside an active lock.
         /// </summary>
-		private void CreateNewPooledConnection()
+		private bool CreateNewPooledConnection()
 		{
-			Driver driver = Driver.Create(settings);
-			idlePool.Enqueue(driver);
-		}
+            Driver driver = null;
+            try
+            {
+                driver = Driver.Create(settings);
+            }
+            catch (Exception)
+            {
+                return false;
+            }
+            idlePool.Enqueue(driver);
+            return true;
+        }
 
 		public void ReleaseConnection(Driver driver)
 		{
@@ -204,7 +214,10 @@
             // or room to make a new connection
             lock (lockObject)
             {
-                return GetPooledConnection();
+                Driver d = GetPooledConnection();
+                if (d == null)
+                    poolGate.Release();
+                return d;
             }
 		}
 	}

Modified: trunk/TestSuite/Source/PoolingTests.cs
===================================================================
--- trunk/TestSuite/Source/PoolingTests.cs	2007-07-24 00:13:06 UTC (rev 808)
+++ trunk/TestSuite/Source/PoolingTests.cs	2007-07-24 00:13:44 UTC (rev 809)
@@ -262,6 +262,72 @@
 
 
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 
 
 
\ No newline at end of file