*/
package org.opendaylight.controller.cluster.datastore;
+import static java.util.Objects.requireNonNull;
+
import akka.actor.ActorSelection;
-import com.google.common.base.Optional;
-import com.google.common.base.Preconditions;
import com.google.common.util.concurrent.FutureCallback;
import com.google.common.util.concurrent.Futures;
+import com.google.common.util.concurrent.MoreExecutors;
import com.google.common.util.concurrent.SettableFuture;
-import org.opendaylight.controller.cluster.datastore.utils.ActorContext;
-import org.opendaylight.controller.sal.core.spi.data.DOMStoreReadTransaction;
-import org.opendaylight.controller.sal.core.spi.data.DOMStoreTransaction;
-import org.opendaylight.controller.sal.core.spi.data.DOMStoreWriteTransaction;
+import java.util.Optional;
+import java.util.SortedSet;
+import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
+import org.opendaylight.controller.cluster.datastore.messages.AbstractRead;
+import org.opendaylight.mdsal.common.api.ReadFailedException;
+import org.opendaylight.mdsal.dom.spi.store.DOMStoreReadTransaction;
+import org.opendaylight.mdsal.dom.spi.store.DOMStoreTransaction;
+import org.opendaylight.mdsal.dom.spi.store.DOMStoreWriteTransaction;
import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
import scala.concurrent.Future;
*/
abstract class LocalTransactionContext extends AbstractTransactionContext {
private final DOMStoreTransaction txDelegate;
-
- LocalTransactionContext(DOMStoreTransaction txDelegate, OperationLimiter limiter) {
- super(limiter);
- this.txDelegate = Preconditions.checkNotNull(txDelegate);
+ private final LocalTransactionReadySupport readySupport;
+ private Exception operationError;
+
+ LocalTransactionContext(final DOMStoreTransaction txDelegate, final TransactionIdentifier identifier,
+ final LocalTransactionReadySupport readySupport) {
+ super(identifier);
+ this.txDelegate = requireNonNull(txDelegate);
+ this.readySupport = readySupport;
}
protected abstract DOMStoreWriteTransaction getWriteDelegate();
protected abstract DOMStoreReadTransaction getReadDelegate();
@Override
- public void writeData(YangInstanceIdentifier path, NormalizedNode<?, ?> data) {
+ @SuppressWarnings("checkstyle:IllegalCatch")
+ public void executeDelete(final YangInstanceIdentifier path, final Boolean havePermit) {
incrementModificationCount();
- getWriteDelegate().write(path, data);
- releaseOperation();
+ if (operationError == null) {
+ try {
+ getWriteDelegate().delete(path);
+ } catch (Exception e) {
+ operationError = e;
+ }
+ }
}
@Override
- public void mergeData(YangInstanceIdentifier path, NormalizedNode<?, ?> data) {
+ @SuppressWarnings("checkstyle:IllegalCatch")
+ public void executeMerge(final YangInstanceIdentifier path, final NormalizedNode<?, ?> data,
+ final Boolean havePermit) {
incrementModificationCount();
- getWriteDelegate().merge(path, data);
- releaseOperation();
+ if (operationError == null) {
+ try {
+ getWriteDelegate().merge(path, data);
+ } catch (Exception e) {
+ operationError = e;
+ }
+ }
}
@Override
- public void deleteData(YangInstanceIdentifier path) {
+ @SuppressWarnings("checkstyle:IllegalCatch")
+ public void executeWrite(final YangInstanceIdentifier path, final NormalizedNode<?, ?> data,
+ final Boolean havePermit) {
incrementModificationCount();
- getWriteDelegate().delete(path);
- releaseOperation();
- }
-
- @Override
- public void readData(YangInstanceIdentifier path, final SettableFuture<Optional<NormalizedNode<?, ?>>> proxyFuture) {
- Futures.addCallback(getReadDelegate().read(path), new FutureCallback<Optional<NormalizedNode<?, ?>>>() {
- @Override
- public void onSuccess(final Optional<NormalizedNode<?, ?>> result) {
- proxyFuture.set(result);
- releaseOperation();
+ if (operationError == null) {
+ try {
+ getWriteDelegate().write(path, data);
+ } catch (Exception e) {
+ operationError = e;
}
-
- @Override
- public void onFailure(final Throwable t) {
- proxyFuture.setException(t);
- releaseOperation();
- }
- });
+ }
}
@Override
- public void dataExists(YangInstanceIdentifier path, final SettableFuture<Boolean> proxyFuture) {
- Futures.addCallback(getReadDelegate().exists(path), new FutureCallback<Boolean>() {
+ public <T> void executeRead(final AbstractRead<T> readCmd, final SettableFuture<T> proxyFuture,
+ final Boolean havePermit) {
+ Futures.addCallback(readCmd.apply(getReadDelegate()), new FutureCallback<T>() {
@Override
- public void onSuccess(final Boolean result) {
+ public void onSuccess(final T result) {
proxyFuture.set(result);
- releaseOperation();
}
@Override
- public void onFailure(final Throwable t) {
- proxyFuture.setException(t);
- releaseOperation();
+ public void onFailure(final Throwable failure) {
+ proxyFuture.setException(failure instanceof Exception
+ ? ReadFailedException.MAPPER.apply((Exception) failure) : failure);
}
- });
+ }, MoreExecutors.directExecutor());
}
private LocalThreePhaseCommitCohort ready() {
logModificationCount();
- acquireOperation();
- return (LocalThreePhaseCommitCohort) getWriteDelegate().ready();
- }
-
- @SuppressWarnings({ "unchecked", "rawtypes" })
- private <T extends Future> T completeOperation(final ActorContext actorContext, final T operationFuture) {
- operationFuture.onComplete(getLimiter(), actorContext.getClientDispatcher());
- return operationFuture;
+ return readySupport.onTransactionReady(getWriteDelegate(), operationError);
}
@Override
- public Future<ActorSelection> readyTransaction() {
+ public Future<ActorSelection> readyTransaction(final Boolean havePermit,
+ final Optional<SortedSet<String>> participatingShardNames) {
final LocalThreePhaseCommitCohort cohort = ready();
- return completeOperation(cohort.getActorContext(), cohort.initiateCoordinatedCommit());
+ return cohort.initiateCoordinatedCommit(participatingShardNames);
}
@Override
- public Future<Object> directCommit() {
+ public Future<Object> directCommit(final Boolean havePermit) {
final LocalThreePhaseCommitCohort cohort = ready();
- return completeOperation(cohort.getActorContext(), cohort.initiateDirectCommit());
- }
-
- @Override
- public boolean supportsDirectCommit() {
- return true;
+ return cohort.initiateDirectCommit();
}
@Override
public void closeTransaction() {
txDelegate.close();
- releaseOperation();
}
}
Code Review / controller.git / blobdiff