On Wed, Jul 8, 2009 at 01:05, Dan Diephouse<[email protected]> wrote:
> Just an FYI, I had a problem with my version of the patch and
> the PredicateDerefQuery.rewrite function. It caused Jackrabbit to get caught
> in an infinite loop on me. Changing the function to just return "this" all
> the time seems to fix things, but I'm not sure if it's the right fix or not.

ah, right. good catch. but the change should rather be:

Index: 
src/main/java/org/apache/jackrabbit/core/query/lucene/PredicateDerefQuery.java
===================================================================
--- 
src/main/java/org/apache/jackrabbit/core/query/lucene/PredicateDerefQuery.java  
    (revision
792113)
+++ 
src/main/java/org/apache/jackrabbit/core/query/lucene/PredicateDerefQuery.java  
    (working
copy)
@@ -140,7 +140,7 @@
         if (cQuery == subQuery) {
             return this;
         } else {
-            return new PredicateDerefQuery(subQuery, refProperty,
nameTest, version, nsMappings);
+            return new PredicateDerefQuery(cQuery, refProperty,
nameTest, version, nsMappings);
         }
     }

I'll fix that.

regards
 marcel

Reply via email to