[
https://issues.apache.org/jira/browse/LUCENE-3570?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Simon Willnauer reassigned LUCENE-3570:
---------------------------------------
Assignee: Simon Willnauer
> Inconsistent comment and code for the method "document(int n, FieldSelector
> fieldSelector)" in the classes
> "org.apache.lucene.index.{IndexReader,FilterIndexReader,SegmentReader}".
> -----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: LUCENE-3570
> URL: https://issues.apache.org/jira/browse/LUCENE-3570
> Project: Lucene - Java
> Issue Type: Bug
> Components: core/index
> Affects Versions: 2.9.3, 2.9.4, 3.0, 3.0.1, 3.0.2, 3.0.3, 3.1, 3.2, 3.3,
> 3.4
> Environment: Platform Independent
> Reporter: SHIN HWEI TAN
> Assignee: Simon Willnauer
> Original Estimate: 5m
> Remaining Estimate: 5m
>
> The method "document" in the "SegmentReader" and "FilterIndexReader" classes
> does not check the "fieldSelector" parameter for "null", whereas the Javadoc
> comment in the abstract superclass "IndexReader" explicitly states the
> parameter "May be null":
> /**
> ...
> * @param fieldSelector The {@link FieldSelector} to use to determine what
> * Fields should be loaded on the Document. May be null, in which
> case
> * all Fields will be loaded.
> ...
> */
> public abstract Document document(int n, FieldSelector fieldSelector)
> throws CorruptIndexException, IOException;
> Suggested Fixes:
> 1. Add code "if (fieldSelector == null) ..." at the beginning of the method
> body in "FilterIndexReader" and "SegmentReader".
> OR
> 2. Remove the part "May be null, in which case all Fields will be loaded"
> from the comment in "IndexReader".
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
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]
