*/
package org.opendaylight.controller.cluster.access.commands;
+import static com.google.common.base.Preconditions.checkState;
+import static java.util.Objects.requireNonNull;
+
import akka.actor.ActorRef;
import com.google.common.annotations.Beta;
-import com.google.common.base.Preconditions;
import java.util.ArrayList;
import java.util.List;
-import javax.annotation.concurrent.NotThreadSafe;
+import org.eclipse.jdt.annotation.NonNull;
import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
-import org.opendaylight.yangtools.concepts.Builder;
import org.opendaylight.yangtools.concepts.Identifiable;
/**
- * A reusable {@link Builder} for creating {@link ModifyTransactionRequest} message instances. Its internal state is
- * reset when {@link #build()} is invoked, hence it can be used to create a sequence of messages.
+ * A reusable builder for creating {@link ModifyTransactionRequest} message instances. Its internal state is reset when
+ * {@link #build()} is invoked, hence it can be used to create a sequence of messages. This class is NOT thread-safe.
*
* @author Robert Varga
*/
@Beta
-@NotThreadSafe
-public final class ModifyTransactionRequestBuilder implements Builder<ModifyTransactionRequest>,
- Identifiable<TransactionIdentifier> {
+public final class ModifyTransactionRequestBuilder implements Identifiable<TransactionIdentifier> {
private final List<TransactionModification> modifications = new ArrayList<>(1);
private final TransactionIdentifier identifier;
private final ActorRef replyTo;
private long sequence;
public ModifyTransactionRequestBuilder(final TransactionIdentifier identifier, final ActorRef replyTo) {
- this.identifier = Preconditions.checkNotNull(identifier);
- this.replyTo = Preconditions.checkNotNull(replyTo);
+ this.identifier = requireNonNull(identifier);
+ this.replyTo = requireNonNull(replyTo);
}
@Override
}
private void checkNotFinished() {
- Preconditions.checkState(protocol == null, "Batch has already been finished");
+ checkState(protocol == null, "Batch has already been finished");
}
public void addModification(final TransactionModification modification) {
checkNotFinished();
- modifications.add(Preconditions.checkNotNull(modification));
+ modifications.add(requireNonNull(modification));
}
public void setSequence(final long sequence) {
- Preconditions.checkState(!haveSequence, "Sequence has already been set");
+ checkState(!haveSequence, "Sequence has already been set");
this.sequence = sequence;
haveSequence = true;
}
return modifications.size();
}
- @Override
- public ModifyTransactionRequest build() {
- Preconditions.checkState(haveSequence, "Request sequence has not been set");
+ public @NonNull ModifyTransactionRequest build() {
+ checkState(haveSequence, "Request sequence has not been set");
final ModifyTransactionRequest ret = new ModifyTransactionRequest(identifier, sequence, replyTo, modifications,
protocol);