lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael McCandless <luc...@mikemccandless.com>
Subject Re: svn commit: r1025929 - /lucene/dev/trunk/lucene/src/java/org/apache/lucene/search/MultiTermQuery.java
Date Thu, 21 Oct 2010 12:34:18 GMT
Whoa, thanks for catching this Uwe!

On Thu, Oct 21, 2010 at 8:21 AM, Uwe Schindler <uwe@thetaphi.de> wrote:

> Why did you change this at all? I would revert or fix as described.

Because the current impl is wasteful, ie it does one too many .next()
calls.  I like your fix (move up the count/cache)... I'll commit.

BTW this count is only for debugging/transparency right?  Ie so you
can see how many terms were visited during MTQ rewrite.

Mike

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message