thomasmueller commented on code in PR #3139:
URL: https://github.com/apache/jackrabbit-oak/pull/3139#discussion_r4075008913


##########
oak-lucene/src/test/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneFunctionIndexCommonTest.java:
##########
@@ -86,4 +91,44 @@ public void shutdownExecutor() {
         executorService.shutdown();
     }
 
+    /**
+     * An index whose only indexed property is a function that evaluates to
+     * null for most nodes must stay "sparse": it should contain a Lucene
+     * document only for the nodes where the function is not null, not one
+     * document per node.
+     */
+    @Test
+    public void sparseIndexForIfExistsFunction() throws CommitFailedException {

Review Comment:
   For all functions, we need to implement them both at indexing time (even the 
parsing of functions, because they need to be in the index definition) as well 
as query time (parsing the XPath + SQL-2, and to send the function to the 
indexes).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to