You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by al...@apache.org on 2013/09/10 14:19:10 UTC

svn commit: r1521458 - in /jackrabbit/oak/trunk/oak-lucene: ./ src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/ src/test/java/org/apache/jackrabbit/oak/jcr/query/ src/test/java/org/apache/jackrabbit/oak/plugins/index/lucene/

Author: alexparvulescu
Date: Tue Sep 10 12:19:09 2013
New Revision: 1521458

URL: http://svn.apache.org/r1521458
Log:
OAK-1007 Switch default Lucene Analyzer to ClassicAnalyzer


Modified:
    jackrabbit/oak/trunk/oak-lucene/pom.xml
    jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndex.java
    jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexConstants.java
    jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryFulltextTest.java
    jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexQueryTest.java

Modified: jackrabbit/oak/trunk/oak-lucene/pom.xml
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/pom.xml?rev=1521458&r1=1521457&r2=1521458&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/pom.xml (original)
+++ jackrabbit/oak/trunk/oak-lucene/pom.xml Tue Sep 10 12:19:09 2013
@@ -91,6 +91,8 @@
       org.apache.jackrabbit.core.query.SimpleQueryTest#testNegativeNumber                            <!-- OAK-327 -->
       org.apache.jackrabbit.core.query.SQL2OrderByTest#testOrderByScore
       org.apache.jackrabbit.oak.jcr.query.QueryFulltextTest#testFulltextRelativeProperty             <!-- OAK-828 -->
+      org.apache.jackrabbit.core.query.FulltextQueryTest#testPredefinedEntityReference             <!-- OAK-XXX -->
+
     </known.issues>
   </properties>
 

Modified: jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndex.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndex.java?rev=1521458&r1=1521457&r2=1521458&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndex.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndex.java Tue Sep 10 12:19:09 2013
@@ -656,7 +656,7 @@ public class LuceneIndex implements Full
             if (hasFulltextToken) {
                 return new WildcardQuery(newFulltextTerm(text));
             } else {
-                return new PrefixQuery(newFulltextTerm(text));
+                return new TermQuery(newFulltextTerm(text));
             }
         } else {
             PhraseQuery pq = new PhraseQuery();

Modified: jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexConstants.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexConstants.java?rev=1521458&r1=1521457&r2=1521458&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexConstants.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexConstants.java Tue Sep 10 12:19:09 2013
@@ -17,7 +17,7 @@
 package org.apache.jackrabbit.oak.plugins.index.lucene;
 
 import org.apache.lucene.analysis.Analyzer;
-import org.apache.lucene.analysis.standard.StandardAnalyzer;
+import org.apache.lucene.analysis.standard.ClassicAnalyzer;
 import org.apache.lucene.analysis.util.CharArraySet;
 import org.apache.lucene.util.Version;
 
@@ -29,7 +29,7 @@ public interface LuceneIndexConstants {
 
     Version VERSION = Version.LUCENE_43;
 
-    Analyzer ANALYZER = new StandardAnalyzer(VERSION, CharArraySet.EMPTY_SET);
+    Analyzer ANALYZER = new ClassicAnalyzer(VERSION, CharArraySet.EMPTY_SET);
 
     /**
      * include only certain property types in the index

Modified: jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryFulltextTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryFulltextTest.java?rev=1521458&r1=1521457&r2=1521458&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryFulltextTest.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/jcr/query/QueryFulltextTest.java Tue Sep 10 12:19:09 2013
@@ -59,7 +59,7 @@ public class QueryFulltextTest extends A
 //                    "where contains([nt:base].[text], cast('hello OR hallo' as string)) */", 
 //                    getResult(q.execute(), "plan"));
             assertEquals("[nt:base] as [nt:base] /* " + 
-                    "aggregate +(:fulltext:hallo* :fulltext:hello*) +text:{* TO *} " + 
+                    "aggregate +(:fulltext:hallo :fulltext:hello) +text:{* TO *} " + 
                     "ft:(text:\"hallo\" OR text:\"hello\") " +
                     "where contains([nt:base].[text], cast('hello OR hallo' as string)) */", 
                     getResult(q.execute(), "plan"));

Modified: jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexQueryTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexQueryTest.java?rev=1521458&r1=1521457&r2=1521458&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexQueryTest.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/test/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexQueryTest.java Tue Sep 10 12:19:09 2013
@@ -156,9 +156,22 @@ public class LuceneIndexQueryTest extend
         root.commit();
 
         StringBuffer stmt = new StringBuffer();
-        stmt.append("//*[jcr:contains(., '/p1/p2')]");
+        stmt.append("//*[jcr:contains(., '/p1/p2*')]");
         assertQuery(stmt.toString(), "xpath", ImmutableList.of("/test/a"));
 
     }
 
+    @Test
+    public void containsPathStrict() throws Exception {
+        root.getTree("/").addChild("matchOnPath");
+        root.getTree("/").addChild("match_on_path");
+        root.commit();
+
+        StringBuffer stmt = new StringBuffer();
+        stmt.append("//*[jcr:contains(., 'match')]");
+        assertQuery(stmt.toString(), "xpath",
+                ImmutableList.of("/match_on_path"));
+
+    }
+
 }