You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "Adrien Grand (Jira)" <ji...@apache.org> on 2021/06/14 06:00:00 UTC

[jira] [Commented] (LUCENE-10000) MultiCollectorManager should have parity with MultiCollector behavior

    [ 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-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org