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%2FLocalTransactionContext.java;h=fdba29c7f7f7ef0d5b02816161b357fb876ca9fa;hb=55a9b9f42a14c56060f74b38f84d444c0fbfecc4;hp=9cf5312c1efccb687a87b05c96612e39d8344a39;hpb=461173ce10acbd8597b8f4c6ccb6c98c7a99447c;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionContext.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionContext.java index 9cf5312c1e..fdba29c7f7 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionContext.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionContext.java @@ -7,15 +7,19 @@ */ package org.opendaylight.controller.cluster.datastore; +import static java.util.Objects.requireNonNull; + import akka.actor.ActorSelection; -import com.google.common.base.Preconditions; import com.google.common.util.concurrent.FutureCallback; import com.google.common.util.concurrent.Futures; import com.google.common.util.concurrent.MoreExecutors; 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.messages.AbstractRead; import org.opendaylight.controller.cluster.datastore.modification.AbstractModification; +import org.opendaylight.mdsal.common.api.ReadFailedException; import org.opendaylight.mdsal.dom.spi.store.DOMStoreReadTransaction; import org.opendaylight.mdsal.dom.spi.store.DOMStoreTransaction; import org.opendaylight.mdsal.dom.spi.store.DOMStoreWriteTransaction; @@ -35,7 +39,7 @@ abstract class LocalTransactionContext extends AbstractTransactionContext { LocalTransactionContext(final DOMStoreTransaction txDelegate, final TransactionIdentifier identifier, final LocalTransactionReadySupport readySupport) { super(identifier); - this.txDelegate = Preconditions.checkNotNull(txDelegate); + this.txDelegate = requireNonNull(txDelegate); this.readySupport = readySupport; } @@ -67,7 +71,8 @@ abstract class LocalTransactionContext extends AbstractTransactionContext { @Override public void onFailure(final Throwable failure) { - proxyFuture.setException(failure); + proxyFuture.setException(failure instanceof Exception + ? ReadFailedException.MAPPER.apply((Exception) failure) : failure); } }, MoreExecutors.directExecutor()); } @@ -78,9 +83,10 @@ abstract class LocalTransactionContext extends AbstractTransactionContext { } @Override - public Future readyTransaction(final Boolean havePermit) { + public Future readyTransaction(final Boolean havePermit, + final Optional> participatingShardNames) { final LocalThreePhaseCommitCohort cohort = ready(); - return cohort.initiateCoordinatedCommit(); + return cohort.initiateCoordinatedCommit(participatingShardNames); } @Override