RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Improve logging for cancellations and errors

Browse source 
FEA: ItEr60S04ValidacionEProbasFuncionaisItEr59S04
This commit is contained in:
Óscar González Fernández 2010-09-12 17:50:54 +02:00
parent a7bc4d43eb
commit 1d34ad3549
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
navalplanner-business/src/main/java/org/navalplanner/business/hibernate/notification/NotBlockingAutoUpdatedSnapshot.java
View file

@ -114,6 +114,8 @@ class NotBlockingAutoUpdatedSnapshot<T> implements IAutoUpdatedSnapshot<T> {
newValue));
return newValue;
}
LOG.info(name + " the ongoing calculation has not been completed. "
+ "Returning previous value");
return previousValue.getValue();
}
@ -121,18 +123,22 @@ class NotBlockingAutoUpdatedSnapshot<T> implements IAutoUpdatedSnapshot<T> {
try {
return ongoingCalculation.get();
} catch (Exception e) {
LOG.error("error creating new snapshot, keeping old value",
e);
LOG.error("error creating new value for " + name
+ ", keeping old value", e);
return previousValue.getValue();
}
}
@Override
void cancel() {
if (ongoingCalculation.isDone() || ongoingCalculation.isCancelled()) {
return;
}
LOG.info(name + " cancelling ongoing future");
try {
ongoingCalculation.cancel(true);
} catch (Exception e) {
LOG.error("error cancelling future", e);
LOG.error("error cancelling future for " + name, e);
}
}
}