You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@directory.apache.org by lu...@apache.org on 2016/12/14 16:31:53 UTC

svn commit: r1774290 - /directory/site/trunk/content/api/user-guide/2.10-ldap-connection-template.mdtext

Author: lucastheisen
Date: Wed Dec 14 16:31:52 2016
New Revision: 1774290

URL: http://svn.apache.org/viewvc?rev=1774290&view=rev
Log:
DIRAPI-287: Documentation is wrong for connection pooling

Modified:
    directory/site/trunk/content/api/user-guide/2.10-ldap-connection-template.mdtext

Modified: directory/site/trunk/content/api/user-guide/2.10-ldap-connection-template.mdtext
URL: http://svn.apache.org/viewvc/directory/site/trunk/content/api/user-guide/2.10-ldap-connection-template.mdtext?rev=1774290&r1=1774289&r2=1774290&view=diff
==============================================================================
--- directory/site/trunk/content/api/user-guide/2.10-ldap-connection-template.mdtext (original)
+++ directory/site/trunk/content/api/user-guide/2.10-ldap-connection-template.mdtext Wed Dec 14 16:31:52 2016
@@ -67,9 +67,12 @@ The connection template manages connecti
     poolConfig.timeBetweenEvictionRunsMillis = -1L;
     poolConfig.whenExhaustedAction = GenericObjectPool.WHEN_EXHAUSTED_BLOCK;
 
+    // could use DefaultPoolableLdapConnectionFactory instead
+    // see javadoc for ValidatingPoolableLdapConnectionFactory
+    // for an explanation
     LdapConnectionTemplate ldapConnectionTemplate = 
         new LdapConnectionTemplate( new LdapConnectionPool(
-            new PoolableLdapConnectionFactory( factory ), poolConfig ) );
+            new ValidatingPoolableLdapConnectionFactory( factory ), poolConfig ) );
 
 This may look like a lot, but most of it is optional and it is the last you will have to think about connections.