jackrabbit-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mreut...@apache.org
Subject svn commit: r959978 - in /jackrabbit/trunk/jackrabbit-core/src: main/java/org/apache/jackrabbit/core/query/lucene/ test/java/org/apache/jackrabbit/core/query/lucene/
Date Fri, 02 Jul 2010 13:29:11 GMT
Author: mreutegg
Date: Fri Jul  2 13:29:11 2010
New Revision: 959978

URL: http://svn.apache.org/viewvc?rev=959978&view=rev
Log:
JCR-2669: Improve reading of cached UUID for given document number

Added:
    jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/IDField.java
  (with props)
    jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/lucene/IDFieldTest.java
  (with props)
Modified:
    jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/NodeIndexer.java

Added: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/IDField.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/IDField.java?rev=959978&view=auto
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/IDField.java
(added)
+++ jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/IDField.java
Fri Jul  2 13:29:11 2010
@@ -0,0 +1,58 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.jackrabbit.core.query.lucene;
+
+import java.io.Reader;
+
+import org.apache.jackrabbit.core.id.NodeId;
+import org.apache.lucene.analysis.TokenStream;
+import org.apache.lucene.document.AbstractField;
+
+/**
+ * <code>IDField</code> implements a lucene field for the id of a node.
+ */
+public class IDField extends AbstractField {
+
+    private static final long serialVersionUID = 3322062255855425638L;
+
+    private final NodeId id;
+
+    public IDField(NodeId id) {
+        this.id = id;
+        this.name = FieldNames.UUID;
+        this.isStored = true;
+        this.isTokenized = false;
+        this.omitNorms = true;
+        this.omitTf = true;
+    }
+
+    public String stringValue() {
+        return id.toString();
+    }
+
+    public Reader readerValue() {
+        return null;
+    }
+
+    public byte[] binaryValue() {
+        return null;
+    }
+
+    public TokenStream tokenStreamValue() {
+        return null;
+    }
+}

Propchange: jackrabbit/trunk/jackrabbit-core/src/main/java/org/apache/jackrabbit/core/query/lucene/IDField.java
------------------------------------------------------------------------------
    svn:eol-style = native

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?rev=959978&r1=959977&r2=959978&view=diff
==============================================================================
--- 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
Fri Jul  2 13:29:11 2010
@@ -213,9 +213,7 @@ public class NodeIndexer {
 
         // special fields
         // UUID
-        doc.add(new Field(
-                FieldNames.UUID, node.getNodeId().toString(),
-                Field.Store.YES, Field.Index.NOT_ANALYZED_NO_NORMS));
+        doc.add(new IDField(node.getNodeId()));
         try {
             // parent UUID
             if (node.getParentId() == null) {

Added: jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/lucene/IDFieldTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/lucene/IDFieldTest.java?rev=959978&view=auto
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/lucene/IDFieldTest.java
(added)
+++ jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/lucene/IDFieldTest.java
Fri Jul  2 13:29:11 2010
@@ -0,0 +1,46 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.jackrabbit.core.query.lucene;
+
+import org.apache.jackrabbit.core.id.NodeId;
+import org.apache.lucene.document.Field;
+
+import junit.framework.TestCase;
+
+/**
+ * <code>IDFieldTest</code>...
+ */
+public class IDFieldTest extends TestCase {
+
+    public void testPerformance() {
+        NodeId id = new NodeId();
+        long time = System.currentTimeMillis();
+        for (int i = 0; i < 1000 * 1000; i++) {
+            new IDField(id);
+        }
+        time = System.currentTimeMillis() - time;
+        System.out.println("IDField: " + time + " ms.");
+
+        time = System.currentTimeMillis();
+        for (int i = 0; i < 1000 * 1000; i++) {
+            new Field(FieldNames.UUID, id.toString(), Field.Store.YES,
+                    Field.Index.NOT_ANALYZED_NO_NORMS);
+        }
+        time = System.currentTimeMillis() - time;
+        System.out.println("Field: " + time + " ms.");
+    }
+}

Propchange: jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/lucene/IDFieldTest.java
------------------------------------------------------------------------------
    svn:eol-style = native



Mime
View raw message