[ https://issues.apache.org/jira/browse/LUCENE-2542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12890575#action_12890575 ]
Shai Erera commented on LUCENE-2542: ------------------------------------ Patch looks good. I think it's ready to commit, but it will take me another day or two until I can get to it. So if someone else wants to commit it, please do. > 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-2542.patch, 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-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org