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%2FLocalTransactionChain.java;h=de98dce5620ba84434cf3d7e679a29fde9d05bca;hp=56466f78408fe24602564f9c62e14a8f5e31add5;hb=28e9832cc97a345d5ceb69262784e5c8fef77e37;hpb=340a2d4c979ac6f8d5adff8bd9e1c9f724e7a164 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 56466f7840..de98dce562 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 @@ -9,7 +9,9 @@ package org.opendaylight.controller.cluster.datastore; import akka.actor.ActorSelection; import com.google.common.base.Preconditions; -import org.opendaylight.controller.cluster.datastore.identifiers.TransactionIdentifier; +import javax.annotation.Nonnull; +import javax.annotation.Nullable; +import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier; import org.opendaylight.controller.sal.core.spi.data.AbstractSnapshotBackedTransactionChain; import org.opendaylight.controller.sal.core.spi.data.DOMStoreReadTransaction; import org.opendaylight.controller.sal.core.spi.data.DOMStoreReadWriteTransaction; @@ -57,7 +59,9 @@ final class LocalTransactionChain extends AbstractSnapshotBackedTransactionChain } @Override - protected DOMStoreThreePhaseCommitCohort createCohort(final SnapshotBackedWriteTransaction transaction, final DataTreeModification modification) { + protected DOMStoreThreePhaseCommitCohort createCohort( + final SnapshotBackedWriteTransaction transaction, + final DataTreeModification modification) { return new LocalChainThreePhaseCommitCohort(transaction, modification); } @@ -76,9 +80,16 @@ final class LocalTransactionChain extends AbstractSnapshotBackedTransactionChain return super.newWriteOnlyTransaction(identifier); } - @SuppressWarnings("unchecked") + @SuppressWarnings({"unchecked", "checkstyle:IllegalCatch"}) @Override - public LocalThreePhaseCommitCohort onTransactionReady(DOMStoreWriteTransaction tx) { + public LocalThreePhaseCommitCohort onTransactionReady(@Nonnull DOMStoreWriteTransaction tx, + @Nullable Exception operationError) { + Preconditions.checkArgument(tx instanceof SnapshotBackedWriteTransaction); + if (operationError != null) { + return new LocalChainThreePhaseCommitCohort((SnapshotBackedWriteTransaction)tx, + operationError); + } + try { return (LocalThreePhaseCommitCohort) tx.ready(); } catch (Exception e) {