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

Uwe Schindler commented on LUCENE-4694:
---------------------------------------

Mike: If you really, really want to move it up:
- Make fields() abstract in IndexReader, only add the final methods that 
delegate to the abstract one
- leave fields() abstract in AtomicReader and CompositeReader, too
- add the MultiFields implementation into BaseCompositeReader -> and only there 
(maybe final)

The current patch is as worse as IndexReader before the split into 
composite/atomic! IndexReader is an abstract class without concrete 
implementations. The final methods only delegate to abstract methods, so they 
are no "implementations".
                
> Add back IndexReader.fields() -> Multi*, or discourage term vectors in some 
> better way
> --------------------------------------------------------------------------------------
>
>                 Key: LUCENE-4694
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4694
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>         Attachments: LUCENE-4694.patch
>
>
> Users can easily get term vectors from any indexreader, but not postings 
> lists. this encourages them to do really slow things: like pulling term 
> vectors for every single document.
> this is really really so much worse than going through multifields or 
> whatever. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to