Nice commit.
I am looking through diffs a bit.
I noticed that fileformats.xml mentions that term vectors store term
positions, but I don't see term positions mentioned in descriptions of
.tv* files.  I don't think they are implemented in this version of TV,
although there is TermPositionVector interface.

Shouldn't there be a 'return true' right in that if block below?

+  public boolean hasVectors() {
+    boolean hasVectors = false;
+    for (int i = 0; i < size(); i++) {
+      if (fieldInfo(i).storeTermVector)
+        hasVectors = true;
+    }
+    return hasVectors;
+  }


Otis
P.S.
I noticed that the old ASL was used in some new classes.  We should use
the current version of the license (version 2).  I believe I put a copy
of it in the root of jakarta-lucene module.
I also believe we will have to replace all ASLs in the code with the
latest version, before releasing 1.4RC1.


--- [EMAIL PROTECTED] wrote:
> cutting     2004/02/20 12:14:56
> 
>   Modified:    .        CHANGES.txt
>                src/java/org/apache/lucene/document Field.java
>                src/java/org/apache/lucene/index
> CompoundFileReader.java
>                         DocumentWriter.java FieldInfo.java
> FieldInfos.java
>                         FieldsReader.java FilterIndexReader.java
>                         IndexReader.java MultiReader.java
>                         SegmentMerger.java SegmentReader.java
>                         SegmentTermDocs.java SegmentTermEnum.java
>                         SegmentTermPositions.java TermDocs.java
>                         TermEnum.java TermInfosWriter.java
>                src/test/org/apache/lucene/index
> TestFilterIndexReader.java
>                src/test/org/apache/lucene/search TestBasics.java
>                xdocs    fileformats.xml
>   Added:       src/java/org/apache/lucene/index
> SegmentTermVector.java
>                         TermFreqVector.java TermPositionVector.java
>                         TermVectorsReader.java TermVectorsWriter.java
>                src/java/org/apache/lucene/search QueryTermVector.java
>                src/java/org/apache/lucene/util StringHelper.java
>                src/test/org/apache/lucene/index DocHelper.java
>                         TestDocumentWriter.java TestFieldInfos.java
>                         TestFieldsReader.java TestMultiReader.java
>                         TestSegmentMerger.java TestSegmentReader.java
>                         TestSegmentTermDocs.java
> TestTermVectorsReader.java
>                         TestTermVectorsWriter.java
>                src/test/org/apache/lucene/search
> TestQueryTermVector.java
>                         TestTermVectors.java
>                src/test/org/apache/lucene/util StringHelperTest.java
>   Log:
>   Added term vector support.
>   
>   Revision  Changes    Path
>   1.74      +5 -1      jakarta-lucene/CHANGES.txt
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/CHANGES.txt.diff?r1=1.73&r2=1.74
>   
>   
>   1.12      +69 -7    
> jakarta-lucene/src/java/org/apache/lucene/document/Field.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/document/Field.java.diff?r1=1.11&r2=1.12
>   
>   
>   1.4       +2 -2     
>
jakarta-lucene/src/java/org/apache/lucene/index/CompoundFileReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/CompoundFileReader.java.diff?r1=1.3&r2=1.4
>   
>   
>   1.9       +51 -15   
> jakarta-lucene/src/java/org/apache/lucene/index/DocumentWriter.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/DocumentWriter.java.diff?r1=1.8&r2=1.9
>   
>   
>   1.2       +5 -1     
> jakarta-lucene/src/java/org/apache/lucene/index/FieldInfo.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/FieldInfo.java.diff?r1=1.1&r2=1.2
>   
>   
>   1.5       +110 -33  
> jakarta-lucene/src/java/org/apache/lucene/index/FieldInfos.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/FieldInfos.java.diff?r1=1.4&r2=1.5
>   
>   
>   1.6       +3 -2     
> jakarta-lucene/src/java/org/apache/lucene/index/FieldsReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/FieldsReader.java.diff?r1=1.5&r2=1.6
>   
>   
>   1.7       +26 -4    
>
jakarta-lucene/src/java/org/apache/lucene/index/FilterIndexReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/FilterIndexReader.java.diff?r1=1.6&r2=1.7
>   
>   
>   1.26      +131 -102 
> jakarta-lucene/src/java/org/apache/lucene/index/IndexReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/IndexReader.java.diff?r1=1.25&r2=1.26
>   
>   
>   1.2       +33 -2    
> jakarta-lucene/src/java/org/apache/lucene/index/MultiReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/MultiReader.java.diff?r1=1.1&r2=1.2
>   
>   
>   1.8       +174 -67  
> jakarta-lucene/src/java/org/apache/lucene/index/SegmentMerger.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/SegmentMerger.java.diff?r1=1.7&r2=1.8
>   
>   
>   1.19      +80 -24   
> jakarta-lucene/src/java/org/apache/lucene/index/SegmentReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/SegmentReader.java.diff?r1=1.18&r2=1.19
>   
>   
>   1.6       +27 -27   
> jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermDocs.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermDocs.java.diff?r1=1.5&r2=1.6
>   
>   
>   1.4       +17 -17   
> jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermEnum.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermEnum.java.diff?r1=1.3&r2=1.4
>   
>   
>   1.6       +1 -1     
>
jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermPositions.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermPositions.java.diff?r1=1.5&r2=1.6
>   
>   
>   1.6       +9 -9     
> jakarta-lucene/src/java/org/apache/lucene/index/TermDocs.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/TermDocs.java.diff?r1=1.5&r2=1.6
>   
>   
>   1.4       +23 -0    
> jakarta-lucene/src/java/org/apache/lucene/index/TermEnum.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/TermEnum.java.diff?r1=1.3&r2=1.4
>   
>   
>   1.4       +5 -11    
> jakarta-lucene/src/java/org/apache/lucene/index/TermInfosWriter.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/TermInfosWriter.java.diff?r1=1.3&r2=1.4
>   
>   
>   1.1                 
>
jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermVector.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/SegmentTermVector.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/java/org/apache/lucene/index/TermFreqVector.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/TermFreqVector.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/java/org/apache/lucene/index/TermPositionVector.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/TermPositionVector.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/java/org/apache/lucene/index/TermVectorsReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/TermVectorsReader.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/java/org/apache/lucene/index/TermVectorsWriter.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/index/TermVectorsWriter.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/java/org/apache/lucene/search/QueryTermVector.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/search/QueryTermVector.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/java/org/apache/lucene/util/StringHelper.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/java/org/apache/lucene/util/StringHelper.java?rev=1.1
>   
>   
>   1.2       +1 -1     
>
jakarta-lucene/src/test/org/apache/lucene/index/TestFilterIndexReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestFilterIndexReader.java.diff?r1=1.1&r2=1.2
>   
>   
>   1.1                 
> jakarta-lucene/src/test/org/apache/lucene/index/DocHelper.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/DocHelper.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/test/org/apache/lucene/index/TestDocumentWriter.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestDocumentWriter.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/test/org/apache/lucene/index/TestFieldInfos.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestFieldInfos.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/test/org/apache/lucene/index/TestFieldsReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestFieldsReader.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/test/org/apache/lucene/index/TestMultiReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestMultiReader.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/test/org/apache/lucene/index/TestSegmentMerger.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestSegmentMerger.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/test/org/apache/lucene/index/TestSegmentReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestSegmentReader.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/test/org/apache/lucene/index/TestSegmentTermDocs.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestSegmentTermDocs.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/test/org/apache/lucene/index/TestTermVectorsReader.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestTermVectorsReader.java?rev=1.1
>   
>   
>   1.1                 
>
jakarta-lucene/src/test/org/apache/lucene/index/TestTermVectorsWriter.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/index/TestTermVectorsWriter.java?rev=1.1
>   
>   
>   1.5       +1 -1     
> jakarta-lucene/src/test/org/apache/lucene/search/TestBasics.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/search/TestBasics.java.diff?r1=1.4&r2=1.5
>   
>   
>   1.1                 
>
jakarta-lucene/src/test/org/apache/lucene/search/TestQueryTermVector.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/search/TestQueryTermVector.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/test/org/apache/lucene/search/TestTermVectors.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/search/TestTermVectors.java?rev=1.1
>   
>   
>   1.1                 
> jakarta-lucene/src/test/org/apache/lucene/util/StringHelperTest.java
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/src/test/org/apache/lucene/util/StringHelperTest.java?rev=1.1
>   
>   
>   1.7       +56 -5     jakarta-lucene/xdocs/fileformats.xml
>   
>  
>
http://cvs.apache.org/viewcvs/jakarta-lucene/xdocs/fileformats.xml.diff?r1=1.6&r2=1.7
>   
>   
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to