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

ASF subversion and git services commented on LUCENE-6272:
---------------------------------------------------------

Commit 1661376 from [~romseygeek] in branch 'dev/trunk'
[ https://svn.apache.org/r1661376 ]

LUCENE-6272: Scorer extends DocIdSetIterator directly

> 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, 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