lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 31841] - [PATCH] MultiSearcher problems with Similarity.docFreq()
Date Tue, 05 Jul 2005 09:19:46 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=31841>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=31841





------- Additional Comments From siberski@learninglab.de  2005-07-05 11:19 -------
Currently, Query.combine() is only called by MultiSearcher and a Highlighter
test case. If I understand it correctly, the intent of this method was to give
Query subclasses a hook to modify the combination algorithm. This hook is not
used by Query subclasses which are part of the Lucene distribution, and I would
be very astonished if someone used it outside of Lucene. Such an implementation
would break now anyway, because BooleanQueries are only handled correctly in the
new Query.combine[] method, but not in mergeBooleanQueries() (which
reimplementations are supposed to call as helper method).

IMHO the cleanest solution would be to remove combine() and
mergeBooleanQueries() from Query and move the query combination logic to
MultiSearcher. Of course, that would be an incompatible API change, and we can't
be 100% sure that no one needs the combine() hook. On the other hand, we now
have useless code in mergeBooleanQueries() and code which is only used by
MultiSearcher in combine(), and that also isn't a desirable situation.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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


Mime
View raw message