You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2012/10/05 00:33:14 UTC
svn commit: r1394303 -
/lucene/dev/branches/ghost_of_4456/lucene/core/src/java/org/apache/lucene/store/CompoundFileWriter.java
Author: rmuir
Date: Thu Oct 4 22:33:13 2012
New Revision: 1394303
URL: http://svn.apache.org/viewvc?rev=1394303&view=rev
Log:
clear nocommits
Modified:
lucene/dev/branches/ghost_of_4456/lucene/core/src/java/org/apache/lucene/store/CompoundFileWriter.java
Modified: lucene/dev/branches/ghost_of_4456/lucene/core/src/java/org/apache/lucene/store/CompoundFileWriter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/ghost_of_4456/lucene/core/src/java/org/apache/lucene/store/CompoundFileWriter.java?rev=1394303&r1=1394302&r2=1394303&view=diff
==============================================================================
--- lucene/dev/branches/ghost_of_4456/lucene/core/src/java/org/apache/lucene/store/CompoundFileWriter.java (original)
+++ lucene/dev/branches/ghost_of_4456/lucene/core/src/java/org/apache/lucene/store/CompoundFileWriter.java Thu Oct 4 22:33:13 2012
@@ -135,7 +135,7 @@ final class CompoundFileWriter implement
}
IOException priorException = null;
IndexOutput entryTableOut = null;
- // nocommit this code should clean up after itself
+ // TODO this code should clean up after itself
// (remove partial .cfs/.cfe)
try {
if (!pendingEntries.isEmpty() || outputTaken.get()) {
@@ -145,8 +145,6 @@ final class CompoundFileWriter implement
// open the compound stream
getOutput();
assert dataOut != null;
- long finalLength = dataOut.getFilePointer();
- assert assertFileLength(finalLength, dataOut);
} catch (IOException e) {
priorException = e;
} finally {
@@ -162,15 +160,6 @@ final class CompoundFileWriter implement
}
}
- private static boolean assertFileLength(long expected, IndexOutput out)
- throws IOException {
- // nocommit move this out!
- out.flush();
- assert expected == out.length() : "expected: " + expected + " was "
- + out.length();
- return true;
- }
-
private final void ensureOpen() {
if (closed) {
throw new AlreadyClosedException("CFS Directory is already closed");