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%2FRemoteTransactionContext.java;h=603fb20a6bd44a22f10ffe407806899ae4565fcb;hb=70b20300990b171743d3c7861e60cdde5d60966f;hp=67916cf1d2c4871fd759c8281c3908f2af5d31e8;hpb=0281535ab08fd795e42df66d25e9a904ff941ad7;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/RemoteTransactionContext.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/RemoteTransactionContext.java index 67916cf1d2..603fb20a6b 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/RemoteTransactionContext.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/RemoteTransactionContext.java @@ -10,9 +10,10 @@ package org.opendaylight.controller.cluster.datastore; import akka.actor.ActorSelection; import akka.dispatch.OnComplete; +import akka.util.Timeout; import com.google.common.base.Preconditions; import com.google.common.util.concurrent.SettableFuture; -import org.opendaylight.controller.cluster.datastore.identifiers.TransactionIdentifier; +import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier; import org.opendaylight.controller.cluster.datastore.messages.AbstractRead; import org.opendaylight.controller.cluster.datastore.messages.BatchedModifications; import org.opendaylight.controller.cluster.datastore.messages.CloseTransaction; @@ -62,8 +63,8 @@ public class RemoteTransactionContext extends AbstractTransactionContext { return actorContext; } - protected Future executeOperationAsync(SerializableMessage msg) { - return completeOperation(actorContext.executeOperationAsync(getActor(), msg.toSerializable())); + protected Future executeOperationAsync(SerializableMessage msg, Timeout timeout) { + return completeOperation(actorContext.executeOperationAsync(getActor(), msg.toSerializable(), timeout)); } @Override @@ -74,11 +75,6 @@ public class RemoteTransactionContext extends AbstractTransactionContext { actorContext.sendOperationAsync(getActor(), new CloseTransaction(getTransactionVersion()).toSerializable()); } - @Override - public boolean supportsDirectCommit() { - return true; - } - @Override public Future directCommit() { LOG.debug("Tx {} directCommit called", getIdentifier()); @@ -109,8 +105,7 @@ public class RemoteTransactionContext extends AbstractTransactionContext { } private BatchedModifications newBatchedModifications() { - return new BatchedModifications(getIdentifier().toString(), getTransactionVersion(), - getIdentifier().getChainId()); + return new BatchedModifications(getIdentifier(), getTransactionVersion()); } private void batchModification(Modification modification) { @@ -146,7 +141,7 @@ public class RemoteTransactionContext extends AbstractTransactionContext { batchedModifications.setReady(ready); batchedModifications.setDoCommitOnReady(doCommitOnReady); batchedModifications.setTotalMessagesSent(++totalBatchedModificationsSent); - sent = executeOperationAsync(batchedModifications); + sent = executeOperationAsync(batchedModifications, actorContext.getTransactionCommitOperationTimeout()); if(ready) { batchedModifications = null; @@ -201,7 +196,8 @@ public class RemoteTransactionContext extends AbstractTransactionContext { } }; - Future future = executeOperationAsync(readCmd.asVersion(getTransactionVersion())); + Future future = executeOperationAsync(readCmd.asVersion(getTransactionVersion()), + actorContext.getOperationTimeout()); future.onComplete(onComplete, actorContext.getClientDispatcher()); }