I guess since attachements seem to be removed, I should file a jira issue for
patches?
Regards Ard
>
> Hello,
>
> First of all,I am not sure what the policy according
> resolving/indication flaws/bugs in Jackrabbit is? Is creating
> a JIRA issue the way to go or mailing the dev list like I do now?
>
> Anyway, there is a flaw in
> MatchAllScorer.calculateDocFilter(). When you have just two
> nodes, with different properties, like "myprop" and
> "myprop2", and you have an xpath String xpath =
> "//[EMAIL PROTECTED], you get both nodes back (to be precise, you'll
> get every node that has a property that startswith "myprop")
>
>
> You can reproduce it by changing the
> SimpleQueryTest.testIsNotNull() a little:
>
> Change
>
> bar.setProperty("text", "the quick brown fox jumps over the
> lazy dog.");
>
> to
>
> bar.setProperty("mytextwhichstartswithmytext", "the quick
> brown fox jumps over the lazy dog.");
>
> Now the test with xpath =
> "//[EMAIL PROTECTED]:primaryType='nt:unstructured' and @mytext]"; fails
> because 2 results. I did test for the trunk and tag 1.3.1 and
> both have the same problem. I have attached
> MatchAllScorer.java.patch in this mail, or should I create a
> JIRA issue for this?
>
> Furthermore I would like to discuss a different
> implementation for the MatchAllScorer, because IMHO the
> current calculateDocFilter() becomes slow pretty fast (see
> bottom email the code part i am referring to: if you have
> 100.000 docs with "mytext" property, and you query [EMAIL PROTECTED]
> the loop below is executed at least 100.000 times). I think
> it might be out of scope for the user-list, or is the
> user-list the place to discuss something like this?
>
> Regards Ard
>
> --------------------------------------------------------------
> ---------
>
> TermEnum terms = reader.terms(new Term(FieldNames.PROPERTIES, field));
> try {
> TermDocs docs = reader.termDocs();
> try {
> while (terms.term() != null
> && terms.term().field() ==
> FieldNames.PROPERTIES
> && terms.term().text().startsWith(field)) {
> docs.seek(terms);
> while (docs.next()) {
> docFilter.set(docs.doc());
> }
> terms.next();
> }
> } finally {
> docs.close();
> }
> } finally {
> terms.close();
> }
>
> --------------------------------------------------------------
> ---------
>
> --
>
> Hippo
> Oosteinde 11
> 1017WT Amsterdam
> The Netherlands
> Tel +31 (0)20 5224466
> -------------------------------------------------------------
> [EMAIL PROTECTED] / [EMAIL PROTECTED] / http://www.hippo.nl
> --------------------------------------------------------------
>
