X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2FNoOpTransactionContext.java;fp=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2FNoOpTransactionContext.java;h=2094cd2f77ff1a8399f88ce0bb4247603c484cb3;hp=a1421429405506d9c4ea1c0f0d4f848bc11c7b78;hb=4379f102fa0c85abf58f60d81fec9c698582fb1a;hpb=ed7166c5ba3a378f46640bb479464580c2b167ba diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/NoOpTransactionContext.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/NoOpTransactionContext.java index a142142940..2094cd2f77 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/NoOpTransactionContext.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/NoOpTransactionContext.java @@ -11,6 +11,7 @@ import akka.actor.ActorSelection; import com.google.common.base.Optional; import com.google.common.util.concurrent.SettableFuture; import org.opendaylight.controller.cluster.datastore.exceptions.NoShardLeaderException; +import org.opendaylight.controller.cluster.datastore.identifiers.TransactionIdentifier; import org.opendaylight.controller.md.sal.common.api.data.DataStoreUnavailableException; import org.opendaylight.controller.md.sal.common.api.data.ReadFailedException; import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; @@ -24,8 +25,8 @@ final class NoOpTransactionContext extends AbstractTransactionContext { private final Throwable failure; - public NoOpTransactionContext(Throwable failure, OperationLimiter limiter) { - super(limiter); + public NoOpTransactionContext(Throwable failure, TransactionIdentifier identifier) { + super(identifier); this.failure = failure; } @@ -42,39 +43,33 @@ final class NoOpTransactionContext extends AbstractTransactionContext { @Override public Future directCommit() { LOG.debug("Tx {} directCommit called, failure: {}", getIdentifier(), failure); - releaseOperation(); return akka.dispatch.Futures.failed(failure); } @Override public Future readyTransaction() { LOG.debug("Tx {} readyTransaction called, failure: {}", getIdentifier(), failure); - releaseOperation(); return akka.dispatch.Futures.failed(failure); } @Override public void deleteData(YangInstanceIdentifier path) { LOG.debug("Tx {} deleteData called path = {}", getIdentifier(), path); - releaseOperation(); } @Override public void mergeData(YangInstanceIdentifier path, NormalizedNode data) { LOG.debug("Tx {} mergeData called path = {}", getIdentifier(), path); - releaseOperation(); } @Override public void writeData(YangInstanceIdentifier path, NormalizedNode data) { LOG.debug("Tx {} writeData called path = {}", getIdentifier(), path); - releaseOperation(); } @Override public void readData(final YangInstanceIdentifier path, SettableFuture>> proxyFuture) { LOG.debug("Tx {} readData called path = {}", getIdentifier(), path); - releaseOperation(); final Throwable t; if (failure instanceof NoShardLeaderException) { @@ -88,7 +83,6 @@ final class NoOpTransactionContext extends AbstractTransactionContext { @Override public void dataExists(YangInstanceIdentifier path, SettableFuture proxyFuture) { LOG.debug("Tx {} dataExists called path = {}", getIdentifier(), path); - releaseOperation(); proxyFuture.setException(new ReadFailedException("Error checking exists for path " + path, failure)); } }