You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2015/09/11 15:32:35 UTC
svn commit: r1702461 - in
/lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs:
lucene40/Lucene40RWSegmentInfoFormat.java
lucene46/Lucene46RWSegmentInfoFormat.java
Author: mikemccand
Date: Fri Sep 11 13:32:34 2015
New Revision: 1702461
URL: http://svn.apache.org/r1702461
Log:
LUCENE-6799: codec should not delete files it claims to have written
Modified:
lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene40/Lucene40RWSegmentInfoFormat.java
lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene46/Lucene46RWSegmentInfoFormat.java
Modified: lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene40/Lucene40RWSegmentInfoFormat.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene40/Lucene40RWSegmentInfoFormat.java?rev=1702461&r1=1702460&r2=1702461&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene40/Lucene40RWSegmentInfoFormat.java (original)
+++ lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene40/Lucene40RWSegmentInfoFormat.java Fri Sep 11 13:32:34 2015
@@ -60,8 +60,6 @@ public final class Lucene40RWSegmentInfo
} finally {
if (!success) {
IOUtils.closeWhileHandlingException(output);
- // TODO: why must we do this? do we not get tracking dir wrapper?
- IOUtils.deleteFilesIgnoringExceptions(si.dir, fileName);
} else {
output.close();
}
Modified: lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene46/Lucene46RWSegmentInfoFormat.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene46/Lucene46RWSegmentInfoFormat.java?rev=1702461&r1=1702460&r2=1702461&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene46/Lucene46RWSegmentInfoFormat.java (original)
+++ lucene/dev/branches/branch_5x/lucene/backward-codecs/src/test/org/apache/lucene/codecs/lucene46/Lucene46RWSegmentInfoFormat.java Fri Sep 11 13:32:34 2015
@@ -62,8 +62,6 @@ public final class Lucene46RWSegmentInfo
} finally {
if (!success) {
IOUtils.closeWhileHandlingException(output);
- // TODO: are we doing this outside of the tracking wrapper? why must SIWriter cleanup like this?
- IOUtils.deleteFilesIgnoringExceptions(si.dir, fileName);
} else {
output.close();
}