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%2FLocalTransactionChain.java;h=fd3e8cd28bdbbd44852e69d2feb5e27c96a0b81f;hb=2dedb8231e13abe55d6b75eb532d23dbe536e168;hp=11b45137c392d37b39e071d60f20afa63c101be1;hpb=461173ce10acbd8597b8f4c6ccb6c98c7a99447c;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionChain.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionChain.java index 11b45137c3..fd3e8cd28b 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionChain.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/LocalTransactionChain.java @@ -7,10 +7,10 @@ */ package org.opendaylight.controller.cluster.datastore; +import static com.google.common.base.Preconditions.checkArgument; +import static java.util.Objects.requireNonNull; + import akka.actor.ActorSelection; -import com.google.common.base.Preconditions; -import javax.annotation.Nonnull; -import javax.annotation.Nullable; import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier; import org.opendaylight.mdsal.dom.spi.store.AbstractSnapshotBackedTransactionChain; import org.opendaylight.mdsal.dom.spi.store.DOMStoreReadTransaction; @@ -18,9 +18,9 @@ import org.opendaylight.mdsal.dom.spi.store.DOMStoreReadWriteTransaction; import org.opendaylight.mdsal.dom.spi.store.DOMStoreThreePhaseCommitCohort; import org.opendaylight.mdsal.dom.spi.store.DOMStoreWriteTransaction; import org.opendaylight.mdsal.dom.spi.store.SnapshotBackedWriteTransaction; -import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTree; -import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeModification; -import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeSnapshot; +import org.opendaylight.yangtools.yang.data.tree.api.DataTreeModification; +import org.opendaylight.yangtools.yang.data.tree.api.DataTreeSnapshot; +import org.opendaylight.yangtools.yang.data.tree.api.ReadOnlyDataTree; /** * Transaction chain instantiated on top of a locally-available DataTree. It does not instantiate @@ -31,15 +31,16 @@ final class LocalTransactionChain extends AbstractSnapshotBackedTransactionChain private static final Throwable ABORTED = new Throwable("Transaction aborted"); private final TransactionChainProxy parent; private final ActorSelection leader; - private final DataTree tree; + private final ReadOnlyDataTree tree; - LocalTransactionChain(final TransactionChainProxy parent, final ActorSelection leader, final DataTree tree) { - this.parent = Preconditions.checkNotNull(parent); - this.leader = Preconditions.checkNotNull(leader); - this.tree = Preconditions.checkNotNull(tree); + LocalTransactionChain(final TransactionChainProxy parent, final ActorSelection leader, + final ReadOnlyDataTree tree) { + this.parent = requireNonNull(parent); + this.leader = requireNonNull(leader); + this.tree = requireNonNull(tree); } - DataTree getDataTree() { + ReadOnlyDataTree getDataTree() { return tree; } @@ -67,25 +68,25 @@ final class LocalTransactionChain extends AbstractSnapshotBackedTransactionChain } @Override - public DOMStoreReadTransaction newReadOnlyTransaction(TransactionIdentifier identifier) { + public DOMStoreReadTransaction newReadOnlyTransaction(final TransactionIdentifier identifier) { return super.newReadOnlyTransaction(identifier); } @Override - public DOMStoreReadWriteTransaction newReadWriteTransaction(TransactionIdentifier identifier) { + public DOMStoreReadWriteTransaction newReadWriteTransaction(final TransactionIdentifier identifier) { return super.newReadWriteTransaction(identifier); } @Override - public DOMStoreWriteTransaction newWriteOnlyTransaction(TransactionIdentifier identifier) { + public DOMStoreWriteTransaction newWriteOnlyTransaction(final TransactionIdentifier identifier) { return super.newWriteOnlyTransaction(identifier); } @SuppressWarnings({"unchecked", "checkstyle:IllegalCatch"}) @Override - public LocalThreePhaseCommitCohort onTransactionReady(@Nonnull DOMStoreWriteTransaction tx, - @Nullable Exception operationError) { - Preconditions.checkArgument(tx instanceof SnapshotBackedWriteTransaction); + public LocalThreePhaseCommitCohort onTransactionReady(final DOMStoreWriteTransaction tx, + final Exception operationError) { + checkArgument(tx instanceof SnapshotBackedWriteTransaction); if (operationError != null) { return new LocalChainThreePhaseCommitCohort((SnapshotBackedWriteTransaction)tx, operationError); @@ -102,23 +103,25 @@ final class LocalTransactionChain extends AbstractSnapshotBackedTransactionChain private class LocalChainThreePhaseCommitCohort extends LocalThreePhaseCommitCohort { - protected LocalChainThreePhaseCommitCohort(SnapshotBackedWriteTransaction transaction, - DataTreeModification modification, Exception operationError) { - super(parent.getActorContext(), leader, transaction, modification, operationError); + protected LocalChainThreePhaseCommitCohort( + final SnapshotBackedWriteTransaction transaction, + final DataTreeModification modification, final Exception operationError) { + super(parent.getActorUtils(), leader, transaction, modification, operationError); } - protected LocalChainThreePhaseCommitCohort(SnapshotBackedWriteTransaction transaction, - Exception operationError) { - super(parent.getActorContext(), leader, transaction, operationError); + protected LocalChainThreePhaseCommitCohort( + final SnapshotBackedWriteTransaction transaction, + final Exception operationError) { + super(parent.getActorUtils(), leader, transaction, operationError); } @Override - protected void transactionAborted(SnapshotBackedWriteTransaction transaction) { + protected void transactionAborted(final SnapshotBackedWriteTransaction transaction) { onTransactionFailed(transaction, ABORTED); } @Override - protected void transactionCommitted(SnapshotBackedWriteTransaction transaction) { + protected void transactionCommitted(final SnapshotBackedWriteTransaction transaction) { onTransactionCommited(transaction); } }