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

Adrien Grand commented on LUCENE-10000:
---------------------------------------

+1

> MultiCollectorManager should have parity with MultiCollector behavior
> ---------------------------------------------------------------------
>
>                 Key: LUCENE-10000
>                 URL: https://issues.apache.org/jira/browse/LUCENE-10000
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/search
>    Affects Versions: main (9.0)
>            Reporter: Greg Miller
>            Assignee: Greg Miller
>            Priority: Minor
>
> The behavior of {{MultiCollectorManager}} and {{MultiCollector}} differ with 
> respect to how they handle {{CollectionTerminationException}} and 
> minCompetitiveScore calls.
> In {{MultiCollector}}, remaining collectors will continue collecting when one 
> or more have thrown a {{CollectionTerminationException}}, but 
> {{MultiCollectorManager}} will allow the exception to propagate and stop 
> collecting against all collectors as soon as one throws (see: LUCENE-6772).
> Also, {{MultiCollector}} properly handles setting of min competitive scores 
> by ensuring the lowest set across all wrapped collectors is used, or by 
> making it a no-op in cases where at least one collector is using 
> {{ScoreMode.COMPLETE}} (see: LUCENE-9402). {{MultiCollectorManager}} will 
> share the same {{Scorable}} across all collectors, which seems like the wrong 
> thing to do.
> Finally, {{MultiCollector}} will cache scores when a {{Scorable}} is shared 
> by more than one collector (see: LUCENE-6263).
> We should update {{MultiCollectorManager}} to use common behavior with 
> {{MultiCollector}} in all these areas. It should be easy to do by delegating 
> to {{MultiCollector}} within {{MultiCollectorManager}}.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to