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%2FShardTransaction.java;h=f747afa7867ac8f7f6434987045315b98bc9cca5;hp=ef9b66a1bcb028233a039203ec02eabc48b5530c;hb=18a4539ad844c05fcd30373efa43f873aca4c142;hpb=d3d2323ad9ad332d71dc6b956e4a3d3daa525902 diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardTransaction.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardTransaction.java index ef9b66a1bc..f747afa786 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardTransaction.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/ShardTransaction.java @@ -8,42 +8,196 @@ package org.opendaylight.controller.cluster.datastore; +import akka.actor.ActorRef; import akka.actor.Props; import akka.actor.UntypedActor; +import akka.event.Logging; +import akka.event.LoggingAdapter; import akka.japi.Creator; +import com.google.common.base.Optional; +import com.google.common.util.concurrent.ListenableFuture; +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.DeleteDataReply; +import org.opendaylight.controller.cluster.datastore.messages.MergeData; +import org.opendaylight.controller.cluster.datastore.messages.MergeDataReply; +import org.opendaylight.controller.cluster.datastore.messages.ReadData; +import org.opendaylight.controller.cluster.datastore.messages.ReadDataReply; +import org.opendaylight.controller.cluster.datastore.messages.ReadyTransaction; +import org.opendaylight.controller.cluster.datastore.messages.ReadyTransactionReply; +import org.opendaylight.controller.cluster.datastore.messages.WriteData; +import org.opendaylight.controller.cluster.datastore.messages.WriteDataReply; +import org.opendaylight.controller.cluster.datastore.modification.CompositeModification; +import org.opendaylight.controller.cluster.datastore.modification.DeleteModification; +import org.opendaylight.controller.cluster.datastore.modification.ImmutableCompositeModification; +import org.opendaylight.controller.cluster.datastore.modification.MergeModification; +import org.opendaylight.controller.cluster.datastore.modification.MutableCompositeModification; +import org.opendaylight.controller.cluster.datastore.modification.WriteModification; import org.opendaylight.controller.sal.core.spi.data.DOMStoreReadWriteTransaction; +import org.opendaylight.controller.sal.core.spi.data.DOMStoreThreePhaseCommitCohort; +import org.opendaylight.yangtools.yang.data.api.InstanceIdentifier; +import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; + +import java.util.concurrent.ExecutionException; /** * The ShardTransaction Actor represents a remote transaction - * + *

* The ShardTransaction Actor delegates all actions to DOMDataReadWriteTransaction - * + *

+ *

* Even though the DOMStore and the DOMStoreTransactionChain implement multiple types of transactions * the ShardTransaction Actor only works with read-write transactions. This is just to keep the logic simple. At this * time there are no known advantages for creating a read-only or write-only transaction which may change over time * at which point we can optimize things in the distributed store as well. + *

+ *

+ * Handles Messages
+ * ----------------
+ *

  • {@link org.opendaylight.controller.cluster.datastore.messages.ReadData} + *
  • {@link org.opendaylight.controller.cluster.datastore.messages.WriteData} + *
  • {@link org.opendaylight.controller.cluster.datastore.messages.MergeData} + *
  • {@link org.opendaylight.controller.cluster.datastore.messages.DeleteData} + *
  • {@link org.opendaylight.controller.cluster.datastore.messages.ReadyTransaction} + *
  • {@link org.opendaylight.controller.cluster.datastore.messages.CloseTransaction} + *

    */ public class ShardTransaction extends UntypedActor { - private final DOMStoreReadWriteTransaction transaction; + private final ActorRef shardActor; + + private final DOMStoreReadWriteTransaction transaction; + + private final MutableCompositeModification modification = + new MutableCompositeModification(); + + private final LoggingAdapter log = + Logging.getLogger(getContext().system(), this); + + public ShardTransaction(DOMStoreReadWriteTransaction transaction, + ActorRef shardActor) { + this.transaction = transaction; + this.shardActor = shardActor; + } + + + public static Props props(final DOMStoreReadWriteTransaction transaction, + final ActorRef shardActor) { + return Props.create(new Creator() { + + @Override + public ShardTransaction create() throws Exception { + return new ShardTransaction(transaction, shardActor); + } + }); + } + + @Override + public void onReceive(Object message) throws Exception { + log.debug("Received message {}", message); + + if (message instanceof ReadData) { + readData((ReadData) message); + } else if (message instanceof WriteData) { + writeData((WriteData) message); + } else if (message instanceof MergeData) { + mergeData((MergeData) message); + } else if (message instanceof DeleteData) { + deleteData((DeleteData) message); + } else if (message instanceof ReadyTransaction) { + readyTransaction((ReadyTransaction) message); + } else if (message instanceof CloseTransaction) { + closeTransaction((CloseTransaction) message); + } else if (message instanceof GetCompositedModification) { + // This is here for testing only + getSender().tell(new GetCompositeModificationReply( + new ImmutableCompositeModification(modification)), getSelf()); + } + } + + private void readData(ReadData message) { + final ActorRef sender = getSender(); + final ActorRef self = getSelf(); + final InstanceIdentifier path = message.getPath(); + final ListenableFuture>> future = + transaction.read(path); + + future.addListener(new Runnable() { + @Override + public void run() { + try { + Optional> optional = future.get(); + if (optional.isPresent()) { + sender.tell(new ReadDataReply(optional.get()), self); + } else { + //TODO : Need to decide what to do here + } + } catch (InterruptedException | ExecutionException e) { + log.error(e, + "An exception happened when reading data from path : " + + path.toString()); + } + + } + }, getContext().dispatcher()); + } + + + private void writeData(WriteData message) { + modification.addModification( + new WriteModification(message.getPath(), message.getData())); + transaction.write(message.getPath(), message.getData()); + getSender().tell(new WriteDataReply(), getSelf()); + } + + private void mergeData(MergeData message) { + modification.addModification( + new MergeModification(message.getPath(), message.getData())); + transaction.merge(message.getPath(), message.getData()); + getSender().tell(new MergeDataReply(), getSelf()); + } + + private void deleteData(DeleteData message) { + modification.addModification(new DeleteModification(message.getPath())); + transaction.delete(message.getPath()); + getSender().tell(new DeleteDataReply(), getSelf()); + } + + private void readyTransaction(ReadyTransaction message) { + DOMStoreThreePhaseCommitCohort cohort = transaction.ready(); + ActorRef cohortActor = getContext().actorOf( + ThreePhaseCommitCohort.props(cohort, shardActor, modification)); + getSender() + .tell(new ReadyTransactionReply(cohortActor.path()), getSelf()); + + } + + private void closeTransaction(CloseTransaction message) { + transaction.close(); + getSender().tell(new CloseTransactionReply(), getSelf()); + } + + + // These classes are in here for test purposes only + + + static class GetCompositedModification { + + } + - public ShardTransaction(DOMStoreReadWriteTransaction transaction) { - this.transaction = transaction; - } + static class GetCompositeModificationReply { + private final CompositeModification modification; - public static Props props(final DOMStoreReadWriteTransaction transaction){ - return Props.create(new Creator(){ + GetCompositeModificationReply(CompositeModification modification) { + this.modification = modification; + } - @Override - public ShardTransaction create() throws Exception { - return new ShardTransaction(transaction); - } - }); - } - @Override - public void onReceive(Object message) throws Exception { - throw new UnsupportedOperationException("onReceive"); - } + public CompositeModification getModification() { + return modification; + } + } }