X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2FShardDataTreeTransactionChain.java;h=f2e9af3656abdc6bf66f36f645e736d15b8d2168;hb=2a035cba6169185444e7074952b5a2f5cebb3bb1;hp=312e11290c88e713827fe2b051d96b667eb633da;hpb=db9a673c114febc785fbd324947ac2c3e3095d06;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardDataTreeTransactionChain.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardDataTreeTransactionChain.java index 312e11290c..f2e9af3656 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardDataTreeTransactionChain.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardDataTreeTransactionChain.java @@ -24,6 +24,7 @@ import org.slf4j.LoggerFactory; @NotThreadSafe final class ShardDataTreeTransactionChain extends ShardDataTreeTransactionParent implements Identifiable { + private static final Logger LOG = LoggerFactory.getLogger(ShardDataTreeTransactionChain.class); private final LocalHistoryIdentifier chainId; private final ShardDataTree dataTree; @@ -54,7 +55,7 @@ final class ShardDataTreeTransactionChain extends ShardDataTreeTransactionParent final DataTreeSnapshot snapshot = getSnapshot(); LOG.debug("Allocated read-only transaction {} snapshot {}", txId, snapshot); - return new ReadOnlyShardDataTreeTransaction(txId, snapshot); + return new ReadOnlyShardDataTreeTransaction(this, txId, snapshot); } ReadWriteShardDataTreeTransaction newReadWriteTransaction(final TransactionIdentifier txId) { @@ -67,20 +68,27 @@ final class ShardDataTreeTransactionChain extends ShardDataTreeTransactionParent void close() { closed = true; + LOG.debug("Closing chain {}", chainId); } @Override - protected void abortTransaction(final AbstractShardDataTreeTransaction transaction) { + void abortFromTransactionActor(final AbstractShardDataTreeTransaction transaction) { if (transaction instanceof ReadWriteShardDataTreeTransaction) { Preconditions.checkState(openTransaction != null, "Attempted to abort transaction %s while none is outstanding", transaction); - LOG.debug("Aborted transaction {}", transaction); + LOG.debug("Aborted open transaction {}", transaction); openTransaction = null; } } @Override - protected ShardDataTreeCohort finishTransaction(final ReadWriteShardDataTreeTransaction transaction) { + void abortTransaction(final AbstractShardDataTreeTransaction transaction, final Runnable callback) { + abortFromTransactionActor(transaction); + dataTree.abortTransaction(transaction, callback); + } + + @Override + ShardDataTreeCohort finishTransaction(final ReadWriteShardDataTreeTransaction transaction) { Preconditions.checkState(openTransaction != null, "Attempted to finish transaction %s while none is outstanding", transaction); @@ -111,7 +119,13 @@ final class ShardDataTreeTransactionChain extends ShardDataTreeTransactionParent } @Override - ShardDataTreeCohort createReadyCohort(final TransactionIdentifier txId, final DataTreeModification modification) { - return dataTree.createReadyCohort(txId, modification); + ShardDataTreeCohort createFailedCohort(final TransactionIdentifier txId, final DataTreeModification mod, + final Exception failure) { + return dataTree.createFailedCohort(txId, mod, failure); + } + + @Override + ShardDataTreeCohort createReadyCohort(final TransactionIdentifier txId, final DataTreeModification mod) { + return dataTree.createReadyCohort(txId, mod); } }