Return-Path: Delivered-To: apmail-jackrabbit-commits-archive@www.apache.org Received: (qmail 86725 invoked from network); 22 May 2007 08:49:19 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 22 May 2007 08:49:19 -0000 Received: (qmail 90384 invoked by uid 500); 22 May 2007 08:49:25 -0000 Delivered-To: apmail-jackrabbit-commits-archive@jackrabbit.apache.org Received: (qmail 90353 invoked by uid 500); 22 May 2007 08:49:25 -0000 Mailing-List: contact commits-help@jackrabbit.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@jackrabbit.apache.org Delivered-To: mailing list commits@jackrabbit.apache.org Received: (qmail 90344 invoked by uid 99); 22 May 2007 08:49:24 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 22 May 2007 01:49:24 -0700 X-ASF-Spam-Status: No, hits=-99.5 required=10.0 tests=ALL_TRUSTED,NO_REAL_NAME X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO eris.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 22 May 2007 01:49:18 -0700 Received: by eris.apache.org (Postfix, from userid 65534) id F2B4A1A981A; Tue, 22 May 2007 01:48:57 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r540492 - in /jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene: NodeIndexer.java SearchIndex.java Date: Tue, 22 May 2007 08:48:57 -0000 To: commits@jackrabbit.apache.org From: mreutegg@apache.org X-Mailer: svnmailer-1.1.0 Message-Id: <20070522084857.F2B4A1A981A@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: mreutegg Date: Tue May 22 01:48:56 2007 New Revision: 540492 URL: http://svn.apache.org/viewvc?view=rev&rev=540492 Log: JCR-931: cluster synchronization NPE Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/NodeIndexer.java jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/NodeIndexer.java URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/NodeIndexer.java?view=diff&rev=540492&r1=540491&r2=540492 ============================================================================== --- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/NodeIndexer.java (original) +++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/NodeIndexer.java Tue May 22 01:48:56 2007 @@ -164,6 +164,12 @@ doc.add(new Field(FieldNames.PARENT, node.getParentId().toString(), Field.Store.YES, Field.Index.UN_TOKENIZED, Field.TermVector.NO)); NodeState parent = (NodeState) stateProvider.getItemState(node.getParentId()); NodeState.ChildNodeEntry child = parent.getChildNodeEntry(node.getNodeId()); + if (child == null) { + // this can only happen when jackrabbit + // is running in a cluster. + throw new RepositoryException("Missing child node entry " + + "for node with id: " + node.getNodeId()); + } String name = NameFormat.format(child.getName(), mappings); doc.add(new Field(FieldNames.LABEL, name, Field.Store.NO, Field.Index.UN_TOKENIZED, Field.TermVector.NO)); } Modified: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java?view=diff&rev=540492&r1=540491&r2=540492 ============================================================================== --- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java (original) +++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/SearchIndex.java Tue May 22 01:48:56 2007 @@ -422,11 +422,11 @@ Document doc = null; try { doc = createDocument(state, getNamespaceMappings()); + retrieveAggregateRoot(state, aggregateRoots); } catch (RepositoryException e) { - log.error("Exception while creating document for node: " + log.warn("Exception while creating document for node: " + state.getNodeId() + ": " + e.toString()); } - retrieveAggregateRoot(state, aggregateRoots); return doc; } }); @@ -451,7 +451,7 @@ try { return createDocument(state, getNamespaceMappings()); } catch (RepositoryException e) { - log.error("Exception while creating document for node: " + log.warn("Exception while creating document for node: " + state.getNodeId() + ": " + e.toString()); } return null;