[ 
https://issues.apache.org/jira/browse/LUCENE-6272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14329644#comment-14329644
 ] 

Alan Woodward commented on LUCENE-6272:
---------------------------------------

Which is obviously different to what everyone else has... Hmpf, I thought I had 
set it up to be consistent with the project.  Will change.

> Scorer should not extend PostingsEnum
> -------------------------------------
>
>                 Key: LUCENE-6272
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6272
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Alan Woodward
>            Priority: Minor
>         Attachments: LUCENE-6272.patch, LUCENE-6272.patch
>
>
> Scorer currently has to implement a whole bunch of methods that are never 
> called.  The only method that Scorer uses in addition to the methods on 
> DocIdSetIterator is freq(), and as currently implemented this means different 
> things on different Scorers:
> * TermScorer returns its underlying termfreq
> * MinShouldMatchScorer returns how many of its subscorers are matching
> * {Exact|Sloppy}PhraseScorer returns how many phrases it has found on a 
> document
> In addition, freq() is never actually called on TermScorer, and it's only 
> used in explain() on the phrase scorers.
> We should make Scorer extend DocIdSetIterator instead.  In place of freq(), 
> Scorer would have a coord() method that by default returns 1, and for boolean 
> scorers returns how many subscorers are matching.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to