[LUCENE-1042] discrepancy in getTermFreqVector-methods

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.3
• Component/s: core/termvectors
• Lucene Fields: New, Patch Available
• Assignee: Grant Ingersoll
• Reporter: Karl Wettin
• Created: 01/Nov/07 19:58
• Updated: 28/Aug/22 11:42
• Resolved: 10/Dec/07 21:49

Description

getTermFreqVector(int, TermVectorMapper) never calls the mapper if there is no term vector, consitent with all the other getTermFreqVector methods that returns null.

getTermFreqVector(int, String, TermVectorMapper) throws an IOException when a field does not contain the term vector.

My suggestion:

Index: src/java/org/apache/lucene/index/SegmentReader.java
===================================================================
--- src/java/org/apache/lucene/index/SegmentReader.java (revision 590149)
+++ src/java/org/apache/lucene/index/SegmentReader.java (working copy)
@@ -648,7 +648,7 @@
     ensureOpen();
     FieldInfo fi = fieldInfos.fieldInfo(field);
     if (fi == null || !fi.storeTermVector || termVectorsReaderOrig == null)
-      throw new IOException("field does not contain term vectors");
+      return;