use setCurrentlyFillingFromInitialLoadOrApplyingOperationReceivedFromMaster in finally clause and not only in exception handler

This commit is contained in:
Axel Uhl committed 2015-07-07 11:16:08 +02:00
1 parent f11fdfe75f
commit a032146c63
1 file changed
+2 -2
@@ -249,10 +249,10 @@ public interface ReplicableWithObjectInputStream<S, O extends OperationWithResul
}
return result;
} catch (Exception e) {
// FIXME why is the following call not performed in a finally clause? What are the consequences of not "cleaning up" in case the method terminates normally?
setCurrentlyFillingFromInitialLoadOrApplyingOperationReceivedFromMaster(false);
logger.log(Level.SEVERE, "apply", e);
throw new RuntimeException(e);
} finally {
setCurrentlyFillingFromInitialLoadOrApplyingOperationReceivedFromMaster(false);
}
}
}