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%2FCompositeDataTreeCohort.java;h=006555f1f8b97d9007cbe0ba19865e2bcb8f3904;hp=07ff936b2f8e87f331e53fb71212a57040b3f9e8;hb=2634ed7138a343f051ff6452ccc7edd3abfc0c3a;hpb=1d3c54640b9fff649fe8d0f57e20d56f8f936cc1
diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/CompositeDataTreeCohort.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/CompositeDataTreeCohort.java
index 07ff936b2f..006555f1f8 100644
--- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/CompositeDataTreeCohort.java
+++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/CompositeDataTreeCohort.java
@@ -13,7 +13,6 @@ import akka.actor.Status.Failure;
import akka.dispatch.ExecutionContexts;
import akka.dispatch.Futures;
import akka.dispatch.Recover;
-import akka.japi.Function;
import akka.pattern.Patterns;
import akka.util.Timeout;
import com.google.common.base.Preconditions;
@@ -21,19 +20,20 @@ import com.google.common.base.Throwables;
import com.google.common.collect.Iterables;
import java.util.Collection;
import java.util.Iterator;
+import java.util.Optional;
import java.util.concurrent.ExecutionException;
import java.util.concurrent.TimeoutException;
+import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
import org.opendaylight.controller.cluster.datastore.DataTreeCohortActor.CanCommit;
import org.opendaylight.controller.cluster.datastore.DataTreeCohortActor.Success;
-import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeCandidateTip;
+import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeCandidate;
import org.opendaylight.yangtools.yang.model.api.SchemaContext;
import scala.concurrent.Await;
import scala.concurrent.Future;
/**
- *
* Composite cohort, which coordinates multiple user-provided cohorts as if it was only one cohort.
- *
+ *
* It tracks current operation and list of cohorts which successfuly finished previous phase in
* case, if abort is necessary to invoke it only on cohort steps which are still active.
*
@@ -70,51 +70,61 @@ class CompositeDataTreeCohort {
*/
COMMITED,
/**
- * Some of cohorts responsed back with unsuccessful message.
- *
+ * Some of cohorts responded back with unsuccessful message.
*/
FAILED,
/**
- *
* Abort message was send to all cohorts which responded with success previously.
- *
*/
ABORTED
}
protected static final Recover EXCEPTION_TO_MESSAGE = new Recover() {
@Override
- public Failure recover(Throwable error) throws Throwable {
+ public Failure recover(final Throwable error) throws Throwable {
return new Failure(error);
}
};
private final DataTreeCohortActorRegistry registry;
- private final String txId;
+ private final TransactionIdentifier txId;
private final SchemaContext schema;
private final Timeout timeout;
private Iterable successfulFromPrevious;
private State state = State.IDLE;
- CompositeDataTreeCohort(DataTreeCohortActorRegistry registry, String txId, SchemaContext schema, Timeout timeout) {
+ CompositeDataTreeCohort(final DataTreeCohortActorRegistry registry, final TransactionIdentifier transactionID,
+ final SchemaContext schema, final Timeout timeout) {
this.registry = Preconditions.checkNotNull(registry);
- this.txId = Preconditions.checkNotNull(txId);
+ this.txId = Preconditions.checkNotNull(transactionID);
this.schema = Preconditions.checkNotNull(schema);
this.timeout = Preconditions.checkNotNull(timeout);
}
- void canCommit(DataTreeCandidateTip tip) throws ExecutionException, TimeoutException {
+ void reset() {
+ switch (state) {
+ case CAN_COMMIT_SENT:
+ case CAN_COMMIT_SUCCESSFUL:
+ case PRE_COMMIT_SENT:
+ case PRE_COMMIT_SUCCESSFUL:
+ case COMMIT_SENT:
+ abort();
+ break;
+ default :
+ break;
+ }
+
+ successfulFromPrevious = null;
+ state = State.IDLE;
+ }
+
+ void canCommit(final DataTreeCandidate tip) throws ExecutionException, TimeoutException {
Collection messages = registry.createCanCommitMessages(txId, tip, schema);
// FIXME: Optimize empty collection list with pre-created futures, containing success.
- Future> canCommitsFuture =
- Futures.traverse(messages, new Function>() {
- @Override
- public Future apply(CanCommit input) {
- return Patterns.ask(input.getCohort(), input, timeout).recover(EXCEPTION_TO_MESSAGE,
- ExecutionContexts.global());
- }
- }, ExecutionContexts.global());
+ Future> canCommitsFuture = Futures.traverse(messages,
+ input -> Patterns.ask(input.getCohort(), input, timeout).recover(EXCEPTION_TO_MESSAGE,
+ ExecutionContexts.global()), ExecutionContexts.global());
changeStateFrom(State.IDLE, State.CAN_COMMIT_SENT);
processResponses(canCommitsFuture, State.CAN_COMMIT_SENT, State.CAN_COMMIT_SUCCESSFUL);
}
@@ -133,25 +143,23 @@ class CompositeDataTreeCohort {
processResponses(commitsFuture, State.COMMIT_SENT, State.COMMITED);
}
- void abort() throws TimeoutException {
- if (successfulFromPrevious != null) {
- sendMesageToSuccessful(new DataTreeCohortActor.Abort(txId));
+ Optional>> abort() {
+ state = State.ABORTED;
+ if (successfulFromPrevious != null && !Iterables.isEmpty(successfulFromPrevious)) {
+ return Optional.of(sendMesageToSuccessful(new DataTreeCohortActor.Abort(txId)));
}
+
+ return Optional.empty();
}
private Future> sendMesageToSuccessful(final Object message) {
- return Futures.traverse(successfulFromPrevious, new Function>() {
-
- @Override
- public Future apply(DataTreeCohortActor.Success cohortResponse) throws Exception {
- return Patterns.ask(cohortResponse.getCohort(), message, timeout);
- }
-
- }, ExecutionContexts.global());
+ return Futures.traverse(successfulFromPrevious, cohortResponse -> Patterns.ask(
+ cohortResponse.getCohort(), message, timeout), ExecutionContexts.global());
}
- private void processResponses(Future> resultsFuture, State currentState, State afterState)
- throws TimeoutException, ExecutionException {
+ @SuppressWarnings("checkstyle:IllegalCatch")
+ private void processResponses(final Future> resultsFuture, final State currentState,
+ final State afterState) throws TimeoutException, ExecutionException {
final Iterable results;
try {
results = Await.result(resultsFuture, timeout.duration());
@@ -177,7 +185,7 @@ class CompositeDataTreeCohort {
changeStateFrom(currentState, afterState);
}
- void changeStateFrom(State expected, State followup) {
+ void changeStateFrom(final State expected, final State followup) {
Preconditions.checkState(state == expected);
state = followup;
}