Author: frm
Date: Tue Dec 5 16:28:10 2017
New Revision: 1817214
URL: http://svn.apache.org/viewvc?rev=1817214&view=rev
Log:
OAK-7031 - Don't delete the temporary file if close() fails
Modified:
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/standby/codec/ResponseDecoder.java
Modified:
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/standby/codec/ResponseDecoder.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/standby/codec/ResponseDecoder.java?rev=1817214&r1=1817213&r2=1817214&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/standby/codec/ResponseDecoder.java
(original)
+++
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/standby/codec/ResponseDecoder.java
Tue Dec 5 16:28:10 2017
@@ -58,14 +58,12 @@ public class ResponseDecoder extends Byt
@Override
public void close() throws IOException {
- try {
- super.close();
- } finally {
- if (Files.deleteIfExists(file.toPath())) {
- log.debug("File {} was deleted", file.getAbsolutePath());
- } else {
- log.debug("Could not delete {}, not found",
file.getAbsoluteFile());
- }
+ super.close();
+
+ if (Files.deleteIfExists(file.toPath())) {
+ log.debug("File {} was deleted", file.getAbsolutePath());
+ } else {
+ log.debug("Could not delete {}, not found",
file.getAbsoluteFile());
}
}