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

Shai Erera commented on LUCENE-5327:
------------------------------------

I think it will look odd to have DV API on IndexSearcher but not IndexReader. I 
can already see people asking "how come". The .document() is irrelevant, since 
the same API exists on IndexReader, we expose it on IndexSearcher for 
convenience reasons only, since users use IndexSearcher to execute a search, 
it's easier for them to call .document() and .termVectors() to build up the 
search results.

Also, I think it's bad if we only expose BDV and NDV on the API.

I think you should just use MultiDV. Irrespective of whether it's slower or 
not, I feel it's wrong to expose those APIs on IndexSearcher if they are not on 
IndexReader.

> Expose getNumericDocValues and getBinaryDocValues at toplevel reader and 
> searcher levels
> ----------------------------------------------------------------------------------------
>
>                 Key: LUCENE-5327
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5327
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/search
>    Affects Versions: 4.5
>            Reporter: John Wang
>         Attachments: patch.diff
>
>
> Expose NumericDocValues and BinaryDocValues in both IndexReader and 
> IndexSearcher apis.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

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

Reply via email to