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;
import org.opendaylight.controller.cluster.datastore.messages.DataExists;
/**
* Actor for a shard write-only transaction.
*
- * @author: syedbahm
+ * @author syedbahm
*/
public class ShardWriteTransaction extends ShardTransaction {
private Exception lastBatchedModificationsException;
private final ReadWriteShardDataTreeTransaction transaction;
- public ShardWriteTransaction(ReadWriteShardDataTreeTransaction transaction, ActorRef shardActor,
- ShardStats shardStats) {
+ public ShardWriteTransaction(final ReadWriteShardDataTreeTransaction transaction, final ActorRef shardActor,
+ final ShardStats shardStats) {
super(shardActor, shardStats, transaction.getIdentifier());
this.transaction = transaction;
}
}
@Override
- public void handleReceive(Object message) {
+ public void handleReceive(final Object message) {
if (message instanceof BatchedModifications) {
batchedModifications((BatchedModifications)message);
} else {
}
@SuppressWarnings("checkstyle:IllegalCatch")
- private void batchedModifications(BatchedModifications batched) {
+ private void batchedModifications(final BatchedModifications batched) {
if (checkClosed()) {
if (batched.isReady()) {
getSelf().tell(PoisonPill.getInstance(), getSelf());
totalBatchedModificationsReceived, batched.getTotalMessagesSent()));
}
- readyTransaction(false, batched.isDoCommitOnReady(), batched.getVersion());
+ readyTransaction(batched);
} else {
getSender().tell(new BatchedModificationsReply(batched.getModifications().size()), getSelf());
}
}
}
- protected final void dataExists(DataExists message) {
+ protected final void dataExists(final DataExists message) {
super.dataExists(transaction, message);
}
- protected final void readData(ReadData message) {
+ protected final void readData(final ReadData message) {
super.readData(transaction, message);
}
private boolean checkClosed() {
- if (transaction.isClosed()) {
+ final boolean ret = transaction.isClosed();
+ if (ret) {
getSender().tell(new akka.actor.Status.Failure(new IllegalStateException(
"Transaction is closed, no modifications allowed")), getSelf());
- return true;
- } else {
- return false;
}
+ return ret;
}
- private void readyTransaction(boolean returnSerialized, boolean doImmediateCommit, short clientTxVersion) {
+ private void readyTransaction(final BatchedModifications batched) {
TransactionIdentifier transactionID = getTransactionId();
LOG.debug("readyTransaction : {}", transactionID);
- getShardActor().forward(new ForwardedReadyTransaction(transactionID, clientTxVersion,
- transaction, doImmediateCommit), getContext());
+ getShardActor().forward(new ForwardedReadyTransaction(transactionID, batched.getVersion(),
+ transaction, batched.isDoCommitOnReady(), batched.getParticipatingShardNames()), getContext());
// The shard will handle the commit from here so we're no longer needed - self-destruct.
getSelf().tell(PoisonPill.getInstance(), getSelf());