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 ch...@apache.org on 2014/10/28 09:43:07 UTC

svn commit: r1634816 - /jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java

Author: chetanm
Date: Tue Oct 28 08:43:07 2014
New Revision: 1634816

URL: http://svn.apache.org/r1634816
Log:
OAK-2005 - Use separate Lucene index for performing property related queries

Removing implemented TODOs
- Support native functions - OAK-2205
- Support for path restrictions - OAK-2211

Modified:
    jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java

Modified: jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java?rev=1634816&r1=1634815&r2=1634816&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexPlanner.java Tue Oct 28 08:43:07 2014
@@ -59,8 +59,6 @@ public class IndexPlanner {
     }
 
     private IndexPlan.Builder getPlanBuilder() {
-        //TODO Support native functions
-
         FullTextExpression ft = filter.getFullTextConstraint();
 
         //IndexPlanner is currently for property indexes only and does not
@@ -109,7 +107,6 @@ public class IndexPlanner {
             }
         }
 
-        //TODO Support for path restrictions
         //TODO Support for property existence queries
         //TODO support for nodeName queries
         return null;
@@ -162,7 +159,6 @@ public class IndexPlanner {
         for (OrderEntry o : sortOrder) {
             //sorting can only be done for known/configured properties
             // and whose types are known
-            //TODO Can sorting be done for array properties
             if (defn.includeProperty(o.getPropertyName()) || defn.isOrdered(o.getPropertyName())
                     && o.getPropertyType() != null
                     && !o.getPropertyType().isArray()) {