svn commit: r1564151 - /manifoldcf/trunk/framework/core/src/main/java/org/apache/manifoldcf/core/throttler/ConnectionBin.java

[kwright]

Author: kwright
Date: Tue Feb  4 02:02:38 2014
New Revision: 1564151

URL: http://svn.apache.org/r1564151
Log:
For now, always signal that a connection should be destroyed rather than 
retained.  This should clean out every pool at polling time, and permit 
connections that share the same connection bins to be allocated fairly.  Part 
of CONNECTORS-872.

Modified:
    
manifoldcf/trunk/framework/core/src/main/java/org/apache/manifoldcf/core/throttler/ConnectionBin.java

Modified: 
manifoldcf/trunk/framework/core/src/main/java/org/apache/manifoldcf/core/throttler/ConnectionBin.java
URL: 
http://svn.apache.org/viewvc/manifoldcf/trunk/framework/core/src/main/java/org/apache/manifoldcf/core/throttler/ConnectionBin.java?rev=1564151&r1=1564150&r2=1564151&view=diff
==============================================================================
--- 
manifoldcf/trunk/framework/core/src/main/java/org/apache/manifoldcf/core/throttler/ConnectionBin.java
 (original)
+++ 
manifoldcf/trunk/framework/core/src/main/java/org/apache/manifoldcf/core/throttler/ConnectionBin.java
 Tue Feb  4 02:02:38 2014
@@ -211,7 +211,10 @@ public class ConnectionBin
       {
         return CONNECTION_DESTROY;
       }
-      return CONNECTION_WITHINBOUNDS;
+      // Hack: always return CONNECTION_DESTROY, because we don't have a good 
way of allocating
+      // connections in multiple pools that share the same connection bin.  
See CONNECTORS-872.
+      //return CONNECTION_WITHINBOUNDS;
+      return CONNECTION_DESTROY;
     }
     return CONNECTION_POOLEMPTY;
   }