lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shai Erera (JIRA)" <j...@apache.org>
Subject [jira] Commented: (LUCENE-2542) TopDocsCollector should be abstract super class that is the real "TopDocsCollector" contract, a subclass should implement the priority-queue logic. e.g. PQTopDocsCollector
Date Sun, 18 Jul 2010 03:04:49 GMT

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

Shai Erera commented on LUCENE-2542:
------------------------------------

I am not sure that we should do this change. The purpose behind TDC was its PQ usage. On top
of it we created TSDC and TFC. What is the benefit of having TDC w/o PQ, just so that we can
have a PQTFC?

Collector is the base class for all, and if someone wants to create a non-PQ TDC he can do
so ... not sure this should be in Lucene core though.

> TopDocsCollector should be abstract super class that is the real "TopDocsCollector" contract,
a subclass should implement the priority-queue logic. e.g. PQTopDocsCollector
> ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: LUCENE-2542
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2542
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Search
>    Affects Versions: 3.0.2
>            Reporter: Woody Anderson
>             Fix For: 4.0
>
>         Attachments: LUCENE-2542.patch, LUCENE_3.0.2-2542.patch
>
>
> TopDocsCollector is both an abstract interface for producing TopDocs as well as a PriorityQueue
based implementation.
> Not all Collectors that could produce TopDocs must use a PriorityQueue, and it would
be advantageous to allow the TopDocsCollector to be an "interface" type abstract class, with
a PQTopDocsCollector sub-class.
> While doing this, it'd be good to clean up the generics uses in these classes. As it's
odd to create a TopFieldCollector and have to case the TopDocs object, when this can be fixed
with generics.

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


Mime
View raw message