[mapguide-commits] r4284 - sandbox/adsk/2.1/Server/src/Common/Manager

svn_mapguide at osgeo.org svn_mapguide at osgeo.org
Tue Oct 6 16:34:18 EDT 2009


Author: brucedechant
Date: 2009-10-06 16:34:17 -0400 (Tue, 06 Oct 2009)
New Revision: 4284

Modified:
   sandbox/adsk/2.1/Server/src/Common/Manager/ServerManager.cpp
Log:
Add conditional compiler setting around ACE reactor connection limit error message as it only applies to Windows.

Modified: sandbox/adsk/2.1/Server/src/Common/Manager/ServerManager.cpp
===================================================================
--- sandbox/adsk/2.1/Server/src/Common/Manager/ServerManager.cpp	2009-10-03 02:44:23 UTC (rev 4283)
+++ sandbox/adsk/2.1/Server/src/Common/Manager/ServerManager.cpp	2009-10-06 20:34:17 UTC (rev 4284)
@@ -872,10 +872,11 @@
     // Increment the total connections
     m_totalConnections++;
 
-    // The limit for the current ACE reactor is 62. 
+#ifdef _WIN32
+    // The limit for the current Windows ACE reactor is 62. 
     // The only reason the value 55 is chosen is because it is slightly under this value and 
     // so we can log an error as the # of active connections approaches the current ACE reactor limit.
-    // TODO: This error logging will need to be revisited if a different ACE reactor is used.
+    // TODO: This error logging will need to be revisited if a different ACE reactor is used on Windows.
     if(m_totalActiveConnections.value() > 55)
     {
         STRING strActiveConnections = L"";
@@ -887,6 +888,9 @@
         // We are dangerously close to exceeding the safe # of active connections for the current ACE reactor
         MG_LOG_ERROR_ENTRY(message.c_str());
     }
+#else
+    // Linux uses a different ACE reactor and so this error message doesn't apply.
+#endif
 }
 
 void MgServerManager::DecrementActiveConnections()



More information about the mapguide-commits mailing list