为什么DirContext.close()没有返回到池的LDAP连接?

时间:2012-08-14 14:53:26

标签: java ldap jndi connection-pooling

我注意到在使用LDAP连接池时,尽管文档为saying otherwise,但在上下文中调用close()似乎并未将其返回到池中。因此,当我尝试从池中获取一个已经达到最大大小的项目时,它会挂起。

我设法将其缩小到最小的情况。即使我相信我确定地在所有相关对象上调用close(),它似乎依赖于垃圾收集来实际将对象返回到池中,这是意料之外的。 为什么会这样?还有其他一些我应该关闭的对象吗?

在下面的代码段中:

  • 我特意将最大池大小设置为1以突出显示问题。
  • 我从池中获得DirContext(第(2)行),尝试将其返回到池中(第(4)行),然后从池中获取另一个(第(6)行)返回相同的返回对象。
  • 相反,第二个请求(第(6)行)挂起对Object.wait()的一些内部调用。我猜它正在等待一个合并的对象变得可用。
  • 如果通过注释(1)关闭池,它不会挂起(但我想要合并!)。
  • 如果我发表评论(3) - 致电SearchResults.next() - 它运作正常。
  • 如果我取消注释第(5)行以强制在“返回池”调用和向池请求新对象之间进行垃圾收集,则它不会挂起。

由于注释掉第(3)行会导致问题消失,或许我没有正确关闭它的返回值,并且它正在保持池化连接打开。但是,方法results.next()在这种情况下返回SearchResult,其中没有close方法,并且在其文档中没有关于如何干净地关闭它的指导。

测试用例:

@Test
public void testHangs() throws NamingException {

    System.setProperty("com.sun.jndi.ldap.connect.pool.debug", "fine");
    System.setProperty("com.sun.jndi.ldap.connect.pool.maxsize", "1");

    Hashtable<String,String> env = new Hashtable<String,String>();
    env.put(Context.INITIAL_CONTEXT_FACTORY, "com.sun.jndi.ldap.LdapCtxFactory");
    env.put(Context.SECURITY_PRINCIPAL, user);
    env.put(Context.SECURITY_CREDENTIALS, passwd);
    env.put(Context.SECURITY_AUTHENTICATION, "simple");
    env.put(Context.PROVIDER_URL, ldapUrl);

    // use a connection pool
    env.put("com.sun.jndi.ldap.connect.pool", "true"); // -----------------  (1)

    // get a context from the pool.
    DirContext context = new InitialDirContext(env); // -------------------- (2)
    NamingEnumeration<SearchResult> results = context.search("", query, getSC());
    // obviously the next two lines would normally be in a 
    // while(results.hasMore()) { ... = results.next(); } loop.
    assertTrue(results.hasMore()); // this is only a problem when there are some results.
    results.next(); // ----------------------------------------------------- (3)

    // ensure the context is returned to the pool.
    results.close();
    context.close(); // ---------------------------------------------------- (4)

    //System.gc(); // ------------------------------------------------------ (5)

    new InitialDirContext(env);  // hangs here! ---------------------------- (6)
}

使用代码,我的控制台显示:

Create com.sun.jndi.ldap.LdapClient@1a7bf11[ldapad:389]
Use com.sun.jndi.ldap.LdapClient@1a7bf11

如果我强制使用GC,我还会看到:

Release com.sun.jndi.ldap.LdapClient@93dee9 <-- on GC
Use com.sun.jndi.ldap.LdapClient@93dee9     <-- on new InitialDirContext

2 个答案:

答案 0 :(得分:8)

经过一番调查后,我发现LDAP连接没有返回到池中,因为SearchResult对象包含对LdapCtx对象的引用。

如果您更换

results.next();

使用以下

SeachResult ob = results.next();
((Context)ob.getObject()).close();

连接将正确返回池中。这似乎是默认实现中的一个错误。

使用Spring LDAPTemplate时,问题不存在,因为它在关闭LdapCtx的环境中提供了一个自定义的“java.naming.factory.object”,作为构建SearchResult过程的一部分。通过将Spring LDAP库添加到类路径并将以下内容添加到InitialContext

,可以很容易地实现这一点。

java.naming.factory.object = org.springframework.ldap.core.support.DefaultDirObjectFactory

完成此操作后,SearchResult持有的对象将从 com.sun.jndi.ldap.LdapCtx:com.sun.jndi.ldap.LdapCtx 更改为 org.springframework.ldap .core.DirContextAdapter DefaultDirObjectFactory 类负责创建DirContextAdapter,并在将 DirContextAdapter 返回到 DirectoryManager 之前关闭LdapCtx。这是 DefaultDirObjectFactory

中的finally块
finally {
        // It seems that the object supplied to the obj parameter is a
        // DirContext instance with reference to the same Ldap connection as
        // the original context. Since it is not the same instance (that's
        // the nameCtx parameter) this one really needs to be closed in
        // order to correctly clean up and return the connection to the pool
        // when we're finished with the surrounding operation.
        if (obj instanceof Context) {

            Context ctx = (Context) obj;
            try {
                ctx.close();
            }
            catch (Exception e) {
                // Never mind this
            }

        }
    }

答案 1 :(得分:4)

更改SearchControls对象,使returningObjFlag属性为false。您通常不需要对象本身,只需要其nameInNamespace及其属性。如果要创建或修改子上下文,则只需要对象本身。