0

有时当我在一个函数中使用多个 Modeshape 动作时，我会收到此错误：

javax.jcr.RepositoryException：ID 为“060742fc6”的会话已关闭，无法再使用。

我在网上找不到任何对此的解释。这就是我所说的：

myFunction( service.doSomething ( service.getStuff ( id, "en_EN" ).getPath() ) );

做某事，getStuff：

@Interceptors({Foo.class, TraceInterceptor.class})
@Override
public Node doSomething(final String bar) throws RepositoryException {

    return modeshape.execute(new JcrHandler<Node>() {

        @Override
        public Node execute(Session session) throws RepositoryException {
            return session.getNode(bar);
        }
    });
}    

@Interceptors(TraceInterceptor.class)
@Override
public ObjectExtended getStuff(final String query, final String language)
    throws RepositoryException {
    return modeshape.execute(new JcrHandler<ObjectExtended>() {

    @Override
    public ObjectExtendedexecute(Session session) 
        throws RepositoryException {
        QueryManager queryManager = session.getWorkspace().getQueryManager();

        ObjectExtendeditem = null;

        String queryWrapped = 
            "select * from [th:this] as c where name(c)='lang_" 
            + language + "' and c.[th:mylabel] "
                    + "= '" + queryStr + "'";
        LOGGER.debug("Query: " + queryWrapped);
        Query query =
            queryManager.createQuery(queryWrapped,Query.JCR_SQL2);

        QueryResult result = query.execute();
        NodeIterator iter = result.getNodes();
        while (iter.hasNext()) {
            Node node = iter.nextNode().getParent();

            if (node.isNodeType("th:term")) {
                item = new ObjectExtended();
                item.setLabel(getLabel(language, node));
                item.setPath(node.getPath());
            }
        }
        return item;
    }
    });
}

请问为什么会这样？我究竟做错了什么？

4

1 回答 1

1

该错误消息意味着两件事之一：要么正在关闭存储库，要么Session.logout()正在调用该方法。

上面的代码都没有显示你的会话是如何被管理的，你也没有说你是否使用了一个框架。但是我怀疑您以某种方式持有 Session 太久（可能在您的框架关闭会话之后），或者 Session 泄漏到多个线程，并且一个线程在另一个线程关闭它之后尝试使用它。

后者可能是一个真正的问题：虽然将单个Session实例从一个线程传递到另一个线程是可以的（只要原始线程不再使用它），但根据 JCR 2.0 规范，Session实例不是线程安全的，不应被并发使用多个线程。

如果您在代码中创建 Session，通常最好使用try-finally块：

Session session = null;
try {
    session = ... // acquire the session
    // use the session, including 0 or more calls to 'save()'
} catch ( RepositoryException e ) {
    // handle it
} finally {
   if ( session != null ) {
       try {
           session.logout();
       } finally {
           session = null;
       }
    }
}

请注意，logout()不抛出 a RepositoryException，因此上述形式通常效果很好。当然，如果您知道您session稍后不会在该方法中使用，则不需要内部try-finally来使引用为空session：

Session session = null;
try {
    session = ... // acquire the session
    // use the session, including 0 or more calls to 'save()'
} catch ( RepositoryException e ) {
    // handle it
} finally {
   if ( session != null ) session.logout();
}

这种逻辑很容易被封装。

于 2013-10-31T12:05:47.203 回答