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 2017/10/09 06:59:03 UTC

svn commit: r1811537 - /jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/property/HybridPropertyIndexLookup.java

Author: chetanm
Date: Mon Oct  9 06:59:03 2017
New Revision: 1811537

URL: http://svn.apache.org/viewvc?rev=1811537&view=rev
Log:
OAK-6535 - Synchronous Lucene Property Indexes

Log the paths returned by property index query

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

Modified: jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/property/HybridPropertyIndexLookup.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/property/HybridPropertyIndexLookup.java?rev=1811537&r1=1811536&r2=1811537&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/property/HybridPropertyIndexLookup.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/property/HybridPropertyIndexLookup.java Mon Oct  9 06:59:03 2017
@@ -31,6 +31,8 @@ import org.apache.jackrabbit.oak.plugins
 import org.apache.jackrabbit.oak.plugins.index.property.strategy.UniqueEntryStoreStrategy;
 import org.apache.jackrabbit.oak.spi.query.Filter;
 import org.apache.jackrabbit.oak.spi.state.NodeState;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import static com.google.common.collect.Iterables.transform;
 import static org.apache.jackrabbit.oak.commons.PathUtils.isAbsolute;
@@ -40,6 +42,7 @@ import static org.apache.jackrabbit.oak.
 import static org.apache.jackrabbit.oak.plugins.index.property.PropertyIndexUtil.encode;
 
 public class HybridPropertyIndexLookup {
+    private static final Logger log = LoggerFactory.getLogger(HybridPropertyIndexLookup.class);
     private final String indexPath;
     private final NodeState indexState;
     private final String pathPrefix;
@@ -100,7 +103,15 @@ public class HybridPropertyIndexLookup {
             result = querySimple(filter, indexName, propIndexNode, encodedValues);
         }
 
-        return transform(result, path -> isAbsolute(path) ? path : "/" + path);
+        Iterable<String> paths = transform(result, path -> isAbsolute(path) ? path : "/" + path);
+
+        if (log.isTraceEnabled()) {
+            paths = transform(paths, path -> {
+                log.trace("[{}] {} = {} -> {}", indexPath, propertyName, encodedValues, path);
+                return path;
+            });
+        }
+        return paths;
     }
 
     private static Iterable<String> queryUnique(Filter filter, String indexName, NodeState propIndexRootNode,