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%2FShardWriteTransaction.java;h=3b175a9c05956174896711412474abac649e0141;hp=5bce73f939cd76fe1e4a8e93d882d1e29b50732c;hb=23472d531aca7f695082a3d57719894bfa34d0ac;hpb=80f668c77ef358ba5fbc53b0332597bd2c4844c2 diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardWriteTransaction.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardWriteTransaction.java index 5bce73f939..3b175a9c05 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardWriteTransaction.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardWriteTransaction.java @@ -11,6 +11,7 @@ package org.opendaylight.controller.cluster.datastore; import akka.actor.ActorRef; import akka.actor.PoisonPill; +import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier; import org.opendaylight.controller.cluster.datastore.jmx.mbeans.shard.ShardStats; import org.opendaylight.controller.cluster.datastore.messages.BatchedModifications; import org.opendaylight.controller.cluster.datastore.messages.BatchedModificationsReply; @@ -30,8 +31,8 @@ public class ShardWriteTransaction extends ShardTransaction { private final ReadWriteShardDataTreeTransaction transaction; public ShardWriteTransaction(ReadWriteShardDataTreeTransaction transaction, ActorRef shardActor, - ShardStats shardStats, String transactionID, short clientTxVersion) { - super(shardActor, shardStats, transactionID, clientTxVersion); + ShardStats shardStats) { + super(shardActor, shardStats, transaction.getId()); this.transaction = transaction; } @@ -41,8 +42,7 @@ public class ShardWriteTransaction extends ShardTransaction { } @Override - public void handleReceive(Object message) throws Exception { - + public void handleReceive(Object message) { if (message instanceof BatchedModifications) { batchedModifications((BatchedModifications)message); } else { @@ -75,7 +75,7 @@ public class ShardWriteTransaction extends ShardTransaction { totalBatchedModificationsReceived, batched.getTotalMessagesSent())); } - readyTransaction(false, batched.isDoCommitOnReady()); + readyTransaction(false, batched.isDoCommitOnReady(), batched.getVersion()); } else { getSender().tell(new BatchedModificationsReply(batched.getModifications().size()), getSelf()); } @@ -89,12 +89,12 @@ public class ShardWriteTransaction extends ShardTransaction { } } - protected final void dataExists(DataExists message, final boolean returnSerialized) { - super.dataExists(transaction, message, returnSerialized); + protected final void dataExists(DataExists message) { + super.dataExists(transaction, message); } - protected final void readData(ReadData message, final boolean returnSerialized) { - super.readData(transaction, message, returnSerialized); + protected final void readData(ReadData message) { + super.readData(transaction, message); } private boolean checkClosed() { @@ -106,13 +106,13 @@ public class ShardWriteTransaction extends ShardTransaction { } } - private void readyTransaction(boolean returnSerialized, boolean doImmediateCommit) { - String transactionID = getTransactionID(); + private void readyTransaction(boolean returnSerialized, boolean doImmediateCommit, short clientTxVersion) { + TransactionIdentifier transactionID = getTransactionID(); LOG.debug("readyTransaction : {}", transactionID); - getShardActor().forward(new ForwardedReadyTransaction(transactionID, getClientTxVersion(), - transaction, returnSerialized, doImmediateCommit), getContext()); + getShardActor().forward(new ForwardedReadyTransaction(transactionID, clientTxVersion, + transaction, doImmediateCommit), getContext()); // The shard will handle the commit from here so we're no longer needed - self-destruct. getSelf().tell(PoisonPill.getInstance(), getSelf());