@Override
public ListenableFuture<Boolean> canCommit() {
- DataTreeModification modification = dataTreeModification();
+ DataTreeModification modification = getDataTreeModification();
try {
dataTree.getDataTree().validate(modification);
LOG.trace("Transaction {} validated", transaction);
@Override
public ListenableFuture<Void> preCommit() {
try {
- candidate = dataTree.getDataTree().prepare(dataTreeModification());
+ candidate = dataTree.getDataTree().prepare(getDataTreeModification());
/*
* FIXME: this is the place where we should be interacting with persistence, specifically by invoking
* persist on the candidate (which gives us a Future).
LOG.trace("Transaction {} prepared candidate {}", transaction, candidate);
return VOID_FUTURE;
} catch (Exception e) {
- LOG.debug("Transaction {} failed to prepare", transaction, e);
+ if(LOG.isTraceEnabled()) {
+ LOG.trace("Transaction {} failed to prepare", transaction, e);
+ } else {
+ LOG.error("Transaction failed to prepare", e);
+ }
return Futures.immediateFailedFuture(e);
}
}
- private DataTreeModification dataTreeModification() {
+ @Override
+ DataTreeModification getDataTreeModification() {
DataTreeModification dataTreeModification = transaction;
if(transaction instanceof PruningDataTreeModification){
dataTreeModification = ((PruningDataTreeModification) transaction).getResultingModification();
try {
dataTree.getDataTree().commit(candidate);
} catch (Exception e) {
- LOG.error("Transaction {} failed to commit", transaction, e);
+ if(LOG.isTraceEnabled()) {
+ LOG.trace("Transaction {} failed to commit", transaction, e);
+ } else {
+ LOG.error("Transaction failed to commit", e);
+ }
return Futures.immediateFailedFuture(e);
}