You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@lucene.apache.org by sh...@apache.org on 2011/05/26 11:24:34 UTC

svn commit: r1127840 - /lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/FreqProxTermsWriter.java

Author: shaie
Date: Thu May 26 09:24:34 2011
New Revision: 1127840

URL: http://svn.apache.org/viewvc?rev=1127840&view=rev
Log:
LUCENE-3144: consumer.finish() accidentally left outside 'finally'

Modified:
    lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/FreqProxTermsWriter.java

Modified: lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/FreqProxTermsWriter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/FreqProxTermsWriter.java?rev=1127840&r1=1127839&r2=1127840&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/FreqProxTermsWriter.java (original)
+++ lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/index/FreqProxTermsWriter.java Thu May 26 09:24:34 2011
@@ -135,8 +135,8 @@ final class FreqProxTermsWriter extends 
         perThread.termsHashPerThread.reset(true);
       }
     } finally {
+      consumer.finish();
     }
-    consumer.finish();
   }
 
   private byte[] payloadBuffer;