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%2FShardWriteTransaction.java;h=e993e4b55ccd8c18cd427f0a45068002c1efc54d;hb=057b787289f7b909d7013c22ac73a1c91c860af8;hp=91e578b46d1f6de8f2e841f7d7f459541a9c3bdb;hpb=f3bc7a6b7d0326e5594604cdc144b967c2a9cdb4;p=controller.git 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 91e578b46d..9c304ecdad 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 @@ -1,75 +1,123 @@ /* + * Copyright (c) 2014 Cisco Systems, Inc. and others. All rights reserved. + * Copyright (c) 2015 Brocade Communications Systems, Inc. and others. All rights reserved. * - * Copyright (c) 2014 Cisco Systems, Inc. and others. All rights reserved. - * - * This program and the accompanying materials are made available under the - * terms of the Eclipse Public License v1.0 which accompanies this distribution, - * and is available at http://www.eclipse.org/legal/epl-v10.html - * + * This program and the accompanying materials are made available under the + * terms of the Eclipse Public License v1.0 which accompanies this distribution, + * and is available at http://www.eclipse.org/legal/epl-v10.html */ package org.opendaylight.controller.cluster.datastore; import akka.actor.ActorRef; import akka.actor.PoisonPill; -import akka.event.Logging; -import akka.event.LoggingAdapter; -import org.opendaylight.controller.cluster.datastore.messages.CloseTransaction; -import org.opendaylight.controller.cluster.datastore.messages.CloseTransactionReply; -import org.opendaylight.controller.cluster.datastore.messages.DeleteData; -import org.opendaylight.controller.cluster.datastore.messages.MergeData; -import org.opendaylight.controller.cluster.datastore.messages.ReadyTransaction; -import org.opendaylight.controller.cluster.datastore.messages.WriteData; -import org.opendaylight.controller.sal.core.spi.data.DOMStoreTransactionChain; -import org.opendaylight.controller.sal.core.spi.data.DOMStoreWriteTransaction; -import org.opendaylight.yangtools.yang.model.api.SchemaContext; +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; +import org.opendaylight.controller.cluster.datastore.messages.ForwardedReadyTransaction; +import org.opendaylight.controller.cluster.datastore.messages.ReadData; +import org.opendaylight.controller.cluster.datastore.modification.Modification; /** + * Actor for a shard write-only transaction. + * * @author: syedbahm - * Date: 8/6/14 */ public class ShardWriteTransaction extends ShardTransaction { - private final DOMStoreWriteTransaction transaction; - private final LoggingAdapter log = - Logging.getLogger(getContext().system(), this); - public ShardWriteTransaction(DOMStoreWriteTransaction transaction, ActorRef shardActor, SchemaContext schemaContext) { - super( shardActor, schemaContext); - this.transaction = transaction; - - } - - public ShardWriteTransaction(DOMStoreTransactionChain transactionChain, DOMStoreWriteTransaction transaction, ActorRef shardActor, SchemaContext schemaContext) { - super(transactionChain, shardActor, schemaContext); - this.transaction = transaction; - } - - @Override - public void handleReceive(Object message) throws Exception { - if (WriteData.SERIALIZABLE_CLASS.equals(message.getClass())) { - writeData(transaction, WriteData.fromSerializable(message, schemaContext)); - } else if (MergeData.SERIALIZABLE_CLASS.equals(message.getClass())) { - mergeData(transaction, MergeData.fromSerializable(message, schemaContext)); - } else if (DeleteData.SERIALIZABLE_CLASS.equals(message.getClass())) { - deleteData(transaction,DeleteData.fromSerizalizable(message)); - } else if (ReadyTransaction.SERIALIZABLE_CLASS.equals(message.getClass())) { - readyTransaction(transaction,new ReadyTransaction()); - }else { - super.handleReceive(message); + + private int totalBatchedModificationsReceived; + private Exception lastBatchedModificationsException; + private final ReadWriteShardDataTreeTransaction transaction; + + public ShardWriteTransaction(ReadWriteShardDataTreeTransaction transaction, ActorRef shardActor, + ShardStats shardStats) { + super(shardActor, shardStats, transaction.getId()); + this.transaction = transaction; + } + + @Override + protected ReadWriteShardDataTreeTransaction getDOMStoreTransaction() { + return transaction; + } + + @Override + public void handleReceive(Object message) { + if (message instanceof BatchedModifications) { + batchedModifications((BatchedModifications)message); + } else { + super.handleReceive(message); + } + } + + @SuppressWarnings("checkstyle:IllegalCatch") + private void batchedModifications(BatchedModifications batched) { + if (checkClosed()) { + if (batched.isReady()) { + getSelf().tell(PoisonPill.getInstance(), getSelf()); + } + return; + } + + try { + for (Modification modification: batched.getModifications()) { + modification.apply(transaction.getSnapshot()); + } + + totalBatchedModificationsReceived++; + if (batched.isReady()) { + if (lastBatchedModificationsException != null) { + throw lastBatchedModificationsException; + } + + if (totalBatchedModificationsReceived != batched.getTotalMessagesSent()) { + throw new IllegalStateException(String.format( + "The total number of batched messages received %d does not match the number sent %d", + totalBatchedModificationsReceived, batched.getTotalMessagesSent())); + } + + readyTransaction(false, batched.isDoCommitOnReady(), batched.getVersion()); + } else { + getSender().tell(new BatchedModificationsReply(batched.getModifications().size()), getSelf()); + } + } catch (Exception e) { + lastBatchedModificationsException = e; + getSender().tell(new akka.actor.Status.Failure(e), getSelf()); + + if (batched.isReady()) { + getSelf().tell(PoisonPill.getInstance(), getSelf()); + } + } + } + + protected final void dataExists(DataExists message) { + super.dataExists(transaction, message); + } + + protected final void readData(ReadData message) { + super.readData(transaction, message); } - } - - protected void closeTransaction(CloseTransaction message) { - transaction.close(); - getSender().tell(new CloseTransactionReply().toSerializable(), getSelf()); - getSelf().tell(PoisonPill.getInstance(), getSelf()); - } - - /** - * The following method is used in unit testing only - * hence the default scope. - * This is done to test out failure cases. - */ - public void forUnitTestOnlyExplicitTransactionClose() { - transaction.close(); + + private boolean checkClosed() { + if (transaction.isClosed()) { + getSender().tell(new akka.actor.Status.Failure(new IllegalStateException( + "Transaction is closed, no modifications allowed")), getSelf()); + return true; + } else { + return false; + } + } + + private void readyTransaction(boolean returnSerialized, boolean doImmediateCommit, short clientTxVersion) { + TransactionIdentifier transactionID = getTransactionId(); + + LOG.debug("readyTransaction : {}", transactionID); + + 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()); } }