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;h=cee9d784acc46e7170b696daaaca26bcc54e2825;hp=2094cd2f77ff1a8399f88ce0bb4247603c484cb3;hb=ccca30bbb1545643c427fc59c23329c5d49f8d4b;hpb=4379f102fa0c85abf58f60d81fec9c698582fb1a 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 2094cd2f77..cee9d784ac 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 @@ -8,12 +8,14 @@ package org.opendaylight.controller.cluster.datastore; import akka.actor.ActorSelection; -import com.google.common.base.Optional; import com.google.common.util.concurrent.SettableFuture; +import java.util.Optional; +import java.util.SortedSet; +import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier; 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.controller.cluster.datastore.messages.AbstractRead; +import org.opendaylight.mdsal.common.api.DataStoreUnavailableException; +import org.opendaylight.mdsal.common.api.ReadFailedException; import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; import org.slf4j.Logger; @@ -25,7 +27,7 @@ final class NoOpTransactionContext extends AbstractTransactionContext { private final Throwable failure; - public NoOpTransactionContext(Throwable failure, TransactionIdentifier identifier) { + NoOpTransactionContext(final Throwable failure, final TransactionIdentifier identifier) { super(identifier); this.failure = failure; } @@ -36,40 +38,23 @@ final class NoOpTransactionContext extends AbstractTransactionContext { } @Override - public boolean supportsDirectCommit() { - return true; - } - - @Override - public Future directCommit() { - LOG.debug("Tx {} directCommit called, failure: {}", getIdentifier(), failure); + public Future directCommit(final Boolean havePermit) { + LOG.debug("Tx {} directCommit called, failure", getIdentifier(), failure); return akka.dispatch.Futures.failed(failure); } @Override - public Future readyTransaction() { - LOG.debug("Tx {} readyTransaction called, failure: {}", getIdentifier(), failure); + public Future readyTransaction(final Boolean havePermit, + final Optional> participatingShardNamess) { + LOG.debug("Tx {} readyTransaction called, failure", getIdentifier(), failure); return akka.dispatch.Futures.failed(failure); } @Override - public void deleteData(YangInstanceIdentifier path) { - LOG.debug("Tx {} deleteData called path = {}", getIdentifier(), path); - } - - @Override - public void mergeData(YangInstanceIdentifier path, NormalizedNode data) { - LOG.debug("Tx {} mergeData called path = {}", getIdentifier(), path); - } - - @Override - public void writeData(YangInstanceIdentifier path, NormalizedNode data) { - LOG.debug("Tx {} writeData called path = {}", getIdentifier(), path); - } - - @Override - public void readData(final YangInstanceIdentifier path, SettableFuture>> proxyFuture) { - LOG.debug("Tx {} readData called path = {}", getIdentifier(), path); + public void executeRead(final AbstractRead readCmd, final SettableFuture proxyFuture, + final Boolean havePermit) { + LOG.debug("Tx {} executeRead {} called path = {}", getIdentifier(), readCmd.getClass().getSimpleName(), + readCmd.getPath()); final Throwable t; if (failure instanceof NoShardLeaderException) { @@ -77,12 +62,24 @@ final class NoOpTransactionContext extends AbstractTransactionContext { } else { t = failure; } - proxyFuture.setException(new ReadFailedException("Error reading data for path " + path, t)); + proxyFuture.setException(new ReadFailedException("Error executeRead " + readCmd.getClass().getSimpleName() + + " for path " + readCmd.getPath(), t)); + } + + @Override + public void executeDelete(final YangInstanceIdentifier path, final Boolean havePermit) { + LOG.debug("Tx {} executeDelete called path = {}", getIdentifier(), path); + } + + @Override + public void executeMerge(final YangInstanceIdentifier path, final NormalizedNode data, + final Boolean havePermit) { + LOG.debug("Tx {} executeMerge called path = {}", getIdentifier(), path); } @Override - public void dataExists(YangInstanceIdentifier path, SettableFuture proxyFuture) { - LOG.debug("Tx {} dataExists called path = {}", getIdentifier(), path); - proxyFuture.setException(new ReadFailedException("Error checking exists for path " + path, failure)); + public void executeWrite(final YangInstanceIdentifier path, final NormalizedNode data, + final Boolean havePermit) { + LOG.debug("Tx {} executeWrite called path = {}", getIdentifier(), path); } }