lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Grant Ingersoll (JIRA)" <>
Subject [jira] Commented: (LUCENE-2272) PayloadNearQuery has hardwired explanation for 'AveragePayloadFunction'
Date Wed, 24 Feb 2010 13:54:27 GMT


Grant Ingersoll commented on LUCENE-2272:


Couple of comments:
*  The base explain method can't be abstract.   Something like:
public Explanation explain(int docId){
    Explanation result = new Explanation();
    result.setDescription("Unimpl Payload Function Explain");
    return result;
should do the trick
* The changes don't seem thread safe any more since there are now member variables.  It may
still be all right, but have you looked at this aspect?

> PayloadNearQuery has hardwired explanation for 'AveragePayloadFunction'
> -----------------------------------------------------------------------
>                 Key: LUCENE-2272
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Search
>            Reporter: Peter Keegan
>            Assignee: Grant Ingersoll
>         Attachments: payloadfunctin-patch.txt
> The 'explain' method in PayloadNearSpanScorer assumes the AveragePayloadFunction was
used. This patch adds the 'explain' method to the 'PayloadFunction' interface, where the Scorer
can call it. Added unit tests for 'explain' and for {Min,Max}PayloadFunction.

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:
For additional commands, e-mail:

View raw message