lucene-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rm...@apache.org
Subject svn commit: r1200726 - in /lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index: TermVectorsConsumer.java TermVectorsConsumerPerField.java
Date Fri, 11 Nov 2011 03:44:21 GMT
Author: rmuir
Date: Fri Nov 11 03:44:20 2011
New Revision: 1200726

URL: http://svn.apache.org/viewvc?rev=1200726&view=rev
Log:
LUCENE-2621: move fieldInfo.commit to finish()

Modified:
    lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumer.java
    lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumerPerField.java

Modified: lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumer.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumer.java?rev=1200726&r1=1200725&r2=1200726&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumer.java
(original)
+++ lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumer.java
Fri Nov 11 03:44:20 2011
@@ -103,11 +103,6 @@ final class TermVectorsConsumer extends 
     writer.startDocument(numVectorFields);
     for (int i = 0; i < numVectorFields; i++) {
       perFields[i].finishDocument();
-      // nocommit: loop thru the fields and commit field info after all are successful?
-      // or commit each one after its done?
-      
-      // commit the termVectors once successful success - FI will otherwise reset them
-      perFields[i].fieldInfo.commitVectors();
     }
 
     assert lastDocID == docState.docID: "lastDocID=" + lastDocID + " docState.docID=" + docState.docID;

Modified: lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumerPerField.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumerPerField.java?rev=1200726&r1=1200725&r2=1200726&view=diff
==============================================================================
--- lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumerPerField.java
(original)
+++ lucene/dev/branches/lucene2621/lucene/src/java/org/apache/lucene/index/TermVectorsConsumerPerField.java
Fri Nov 11 03:44:20 2011
@@ -158,6 +158,8 @@ final class TermVectorsConsumerPerField 
     }
 
     termsHashPerField.reset();
+    // commit the termVectors once successful success - FI will otherwise reset them
+    fieldInfo.commitVectors();
   }
 
   void shrinkHash() {



Mime
View raw message