Yes, that makes sense, will do that when refactoring this.


On 13 November 2013 11:50, Simon Willnauer <notifications@github.com> wrote:

> In
> src/main/java/org/elasticsearch/common/lucene/search/XBooleanFilter.java:
>
> >      }
> > +
> > +    static boolean iteratorMatch(DocIdSetIterator docIdSetIterator, int target) throws IOException {
>
> I think we should take the iterators that returned NO_MORE_DOCS out of the
> loop but we can do this in a different change?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/elasticsearch/elasticsearch/pull/4144/files#r7622391>
> .
>



-- 
Met vriendelijke groet,

Martijn van Groningen