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=32674>.
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=32674





------- Additional Comments From [EMAIL PROTECTED]  2004-12-13 23:51 -------
Oops, forgot I use Java 1.5 and forgot I had improved this class by taking
advatnage of it.  I don't believe the original version of MaxDisjunctionQuery
that I sent in email sometime ago had this issue, but it seems moot nonetheless
as Paul Elschot's DisjunctionQuery is going into Lucene and can be used to get
the same effect.  This can be done easily if it is factored such that
DisjunctionQuery provides a general mechanism for subclasses to initialize
state, update state, and produce the final score as it combines the subscorers,
while DisjunctionSumQuery overrides this to implement an optimized version that
doesn't require the method calls.

So my recommendation, would be to factor DisjunctionQuery to make overriding for
different combining operators easy and to include a version of
DisjunctionMaxQuery that uses this to implement the MaxDisjunctionQuery scoring
semantics (i.e., combining operator = max plus constant times sum of other
terms).  I'd be happy to write that version of DisjunctionMaxQuery once
DisjunctionQuery comes out in a released version of Lucene if it's not done for
the release.


-- 
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to