X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-dom-broker%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fmd%2Fsal%2Fdom%2Fbroker%2Fimpl%2FDOMForwardedWriteTransaction.java;h=58c91001c4e8887fec5a297507567fd69dc3a649;hp=eeb345e9ce663e7340300f779ab789805be86a7f;hb=2a6aa1775604906755883f810ee9ea6d5f286135;hpb=8eabfd813c5d19bbe9c2ab01ccbff32eeec58419 diff --git a/opendaylight/md-sal/sal-dom-broker/src/main/java/org/opendaylight/controller/md/sal/dom/broker/impl/DOMForwardedWriteTransaction.java b/opendaylight/md-sal/sal-dom-broker/src/main/java/org/opendaylight/controller/md/sal/dom/broker/impl/DOMForwardedWriteTransaction.java index eeb345e9ce..58c91001c4 100644 --- a/opendaylight/md-sal/sal-dom-broker/src/main/java/org/opendaylight/controller/md/sal/dom/broker/impl/DOMForwardedWriteTransaction.java +++ b/opendaylight/md-sal/sal-dom-broker/src/main/java/org/opendaylight/controller/md/sal/dom/broker/impl/DOMForwardedWriteTransaction.java @@ -8,117 +8,157 @@ package org.opendaylight.controller.md.sal.dom.broker.impl; import static com.google.common.base.Preconditions.checkState; +import static java.util.Objects.requireNonNull; -import javax.annotation.concurrent.GuardedBy; - -import org.opendaylight.controller.md.sal.common.api.TransactionStatus; +import com.google.common.base.Supplier; +import com.google.common.util.concurrent.FluentFuture; +import com.google.common.util.concurrent.Futures; +import com.google.common.util.concurrent.ListenableFuture; +import java.util.ArrayList; +import java.util.Collection; +import java.util.Map; +import java.util.concurrent.Future; +import java.util.concurrent.atomic.AtomicReferenceFieldUpdater; import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType; import org.opendaylight.controller.md.sal.dom.api.DOMDataWriteTransaction; import org.opendaylight.controller.sal.core.spi.data.DOMStoreThreePhaseCommitCohort; import org.opendaylight.controller.sal.core.spi.data.DOMStoreWriteTransaction; -import org.opendaylight.yangtools.yang.common.RpcResult; -import org.opendaylight.yangtools.yang.data.api.InstanceIdentifier; +import org.opendaylight.mdsal.common.api.CommitInfo; +import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; - -import com.google.common.base.Preconditions; -import com.google.common.collect.ImmutableList; -import com.google.common.collect.ImmutableMap; -import com.google.common.util.concurrent.ListenableFuture; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; /** - * - * * Read-Write Transaction, which is composed of several - * {@link DOMStoreWriteTransaction} transactions. Subtransaction is selected by + * {@link DOMStoreWriteTransaction} transactions. A sub-transaction is selected by * {@link LogicalDatastoreType} type parameter in: * + *

*

+ * *

* {@link #commit()} will result in invocation of - * {@link DOMDataCommitImplementation#commit(org.opendaylight.controller.md.sal.dom.api.DOMDataWriteTransaction, Iterable)} - * invocation with all {@link org.opendaylight.controller.sal.core.spi.data.DOMStoreThreePhaseCommitCohort} for underlying - * transactions. + * {@link DOMDataCommitImplementation#submit(org.opendaylight.controller.md.sal.dom.api.DOMDataWriteTransaction, + * Iterable)} invocation with all {@link org.opendaylight.controller.sal.core.spi.data.DOMStoreThreePhaseCommitCohort} + * for underlying transactions. * - * @param - * Subtype of {@link DOMStoreWriteTransaction} which is used as + * @param Subtype of {@link DOMStoreWriteTransaction} which is used as * subtransaction. */ +@Deprecated(forRemoval = true) class DOMForwardedWriteTransaction extends AbstractDOMForwardedCompositeTransaction implements DOMDataWriteTransaction { - - @GuardedBy("this") - private DOMDataCommitImplementation commitImpl; - - @GuardedBy("this") - private boolean canceled; - @GuardedBy("this") - private ListenableFuture> commitFuture; - - protected DOMForwardedWriteTransaction(final Object identifier, - final ImmutableMap backingTxs, final DOMDataCommitImplementation commitImpl) { + @SuppressWarnings("rawtypes") + private static final AtomicReferenceFieldUpdater + IMPL_UPDATER = AtomicReferenceFieldUpdater + .newUpdater(DOMForwardedWriteTransaction.class, AbstractDOMForwardedTransactionFactory.class, "commitImpl"); + @SuppressWarnings("rawtypes") + private static final AtomicReferenceFieldUpdater FUTURE_UPDATER + = AtomicReferenceFieldUpdater.newUpdater(DOMForwardedWriteTransaction.class, Future.class, "commitFuture"); + private static final Logger LOG = LoggerFactory.getLogger(DOMForwardedWriteTransaction.class); + private static final Future CANCELLED_FUTURE = Futures.immediateCancelledFuture(); + + /** + * Implementation of real commit. It also acts as an indication that + * the transaction is running -- which we flip atomically using + * {@link #IMPL_UPDATER}. + */ + private volatile AbstractDOMForwardedTransactionFactory commitImpl; + + /** + * Future task of transaction commit. It starts off as null, but is + * set appropriately on {@link #commit()} and {@link #cancel()} via + * {@link AtomicReferenceFieldUpdater#lazySet(Object, Object)}. + * + *

+ * Lazy set is safe for use because it is only referenced to in the + * {@link #cancel()} slow path, where we will busy-wait for it. The + * fast path gets the benefit of a store-store barrier instead of the + * usual store-load barrier. + */ + private volatile Future commitFuture; + + protected DOMForwardedWriteTransaction(final Object identifier, final Map backingTxs, + final AbstractDOMForwardedTransactionFactory commitImpl) { super(identifier, backingTxs); - this.commitImpl = Preconditions.checkNotNull(commitImpl, "commitImpl must not be null."); + this.commitImpl = requireNonNull(commitImpl, "commitImpl must not be null."); } @Override - public void put(final LogicalDatastoreType store, final InstanceIdentifier path, final NormalizedNode data) { - checkNotReady(); + public void put(final LogicalDatastoreType store, final YangInstanceIdentifier path, + final NormalizedNode data) { + checkRunning(commitImpl); getSubtransaction(store).write(path, data); } @Override - public void delete(final LogicalDatastoreType store, final InstanceIdentifier path) { - checkNotReady(); + public void delete(final LogicalDatastoreType store, final YangInstanceIdentifier path) { + checkRunning(commitImpl); getSubtransaction(store).delete(path); } @Override - public void merge(final LogicalDatastoreType store, final InstanceIdentifier path, final NormalizedNode data) { - checkNotReady(); + public void merge(final LogicalDatastoreType store, final YangInstanceIdentifier path, + final NormalizedNode data) { + checkRunning(commitImpl); getSubtransaction(store).merge(path, data); } @Override - public synchronized void cancel() { - checkState(!canceled, "Transaction was canceled."); - if (commitFuture != null) { - // FIXME: Implement cancelation of commit future - // when Broker impl will support cancelation. - throw new UnsupportedOperationException("Not implemented yet."); + public boolean cancel() { + final AbstractDOMForwardedTransactionFactory impl = IMPL_UPDATER.getAndSet(this, null); + if (impl != null) { + LOG.trace("Transaction {} cancelled before submit", getIdentifier()); + FUTURE_UPDATER.lazySet(this, CANCELLED_FUTURE); + closeSubtransactions(); + return true; } - canceled = true; - commitImpl = null; + // The transaction is in process of being submitted or cancelled. Busy-wait + // for the corresponding future. + Future future; + do { + future = commitFuture; + } while (future == null); + + return future.cancel(false); } @Override - public synchronized ListenableFuture> commit() { - checkNotReady(); - - ImmutableList.Builder cohortsBuilder = ImmutableList.builder(); - for (DOMStoreWriteTransaction subTx : getSubtransactions()) { - cohortsBuilder.add(subTx.ready()); - } - ImmutableList cohorts = cohortsBuilder.build(); - commitFuture = commitImpl.commit(this, cohorts); - return commitFuture; + public FluentFuture commit() { + return FluentFuture.from(doCommit(CommitInfo::empty)); } - private void checkNotReady() { - checkNotCanceled(); - checkNotCommited(); - } + @SuppressWarnings("checkstyle:IllegalCatch") + private ListenableFuture doCommit(final Supplier futureValueSupplier) { + final AbstractDOMForwardedTransactionFactory impl = IMPL_UPDATER.getAndSet(this, null); + checkRunning(impl); - private void checkNotCanceled() { - Preconditions.checkState(!canceled, "Transaction was canceled."); - } + final Collection txns = getSubtransactions(); + final Collection cohorts = new ArrayList<>(txns.size()); + + ListenableFuture ret; + try { + for (DOMStoreWriteTransaction txn : txns) { + cohorts.add(txn.ready()); + } - private void checkNotCommited() { - checkState(commitFuture == null, "Transaction was already commited."); + ret = impl.commit(this, cohorts, futureValueSupplier); + } catch (RuntimeException e) { + ret = FluentFuture.from(Futures.immediateFailedFuture( + TransactionCommitFailedExceptionMapper.COMMIT_ERROR_MAPPER.apply(e))); + } + FUTURE_UPDATER.lazySet(this, ret); + return ret; } -} \ No newline at end of file + private void checkRunning(final AbstractDOMForwardedTransactionFactory impl) { + checkState(impl != null, "Transaction %s is no longer running", getIdentifier()); + } +}