[ 
https://issues.apache.org/jira/browse/LUCENE-1169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12566971#action_12566971
 ] 

Eks Dev commented on LUCENE-1169:
---------------------------------

Thank you for fixing it in no time :) But...

I am getting confused with skipping iterators semantics,

is this requirement for the other DocIdSetIterators, of only for scorers 
(should be, I guess)?

iterator.skipTo(iterator.doc()) <=> iterator.next();// is this contract?

if that is the case,  we have another bug in OpenBitSetIterator (border 
condition)

//this is the code in javadoc, "official contract"
  boolean simulatedSkipTo(DocIdSetIterator i, int target) throws IOException {
    do {
      if (!i.next())
    return false;
    } while (target > i.doc());
    return true;
  }

  public void testOpenBitSetBorderCondition() throws IOException {
    OpenBitSet bs = new OpenBitSet();
    bs.set(0);
    DocIdSetIterator i = bs.iterator();

    i.skipTo(i.doc());
    assertEquals(0, i.doc()); //cool, moved to the first legal position

    assertFalse("End of Matcher", i.skipTo(i.doc())); //NOT OK according to the 
javadoc
  }

  public void testOpenBitSetBorderConditionSimulated() throws IOException {
    OpenBitSet bs = new OpenBitSet();
    bs.set(0);
    DocIdSetIterator i = bs.iterator();

    simulatedSkipTo(i, i.doc());
    assertEquals(0, i.doc()); //cool, moved to the first legal position

    assertFalse("End of Matcher", simulatedSkipTo(i, i.doc())); //OK according 
to the javadoc!!
  }


> Search with Filter does not work!
> ---------------------------------
>
>                 Key: LUCENE-1169
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1169
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Search
>            Reporter: Eks Dev
>            Assignee: Michael Busch
>            Priority: Blocker
>         Attachments: lucene-1169.patch, TestFilteredSearch.java
>
>
> See attached JUnitTest, self-explanatory

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to