*/
package org.opendaylight.controller.cluster.raft;
+import static java.util.Objects.requireNonNull;
+
import akka.actor.Cancellable;
-import com.google.common.base.Preconditions;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
import scala.concurrent.duration.FiniteDuration;
/**
* An abstract class that implements a Runnable operation with a timer such that if the run method isn't
* invoked within a timeout period, the operation is cancelled via {@link #doCancel}.
+ *
* <p>
* <b>Note:</b> this class is not thread safe and is intended for use only within the context of the same
* actor that's passed on construction. The run method must be called on this actor's thread dispatcher as it
private final Cancellable cancelTimer;
private boolean canRun = true;
- TimedRunnable(FiniteDuration timeout, RaftActor actor) {
- Preconditions.checkNotNull(timeout);
- Preconditions.checkNotNull(actor);
- cancelTimer = actor.getContext().system().scheduler().scheduleOnce(timeout, actor.self(), new Runnable() {
- @Override
- public void run() {
- cancel();
- }
- }, actor.getContext().system().dispatcher(), actor.self());
+ @SuppressFBWarnings(value = "MC_OVERRIDABLE_METHOD_CALL_IN_CONSTRUCTOR",
+ justification = "https://github.com/spotbugs/spotbugs/issues/1867")
+ TimedRunnable(final FiniteDuration timeout, final RaftActor actor) {
+ cancelTimer = requireNonNull(actor).getContext().system().scheduler()
+ .scheduleOnce(requireNonNull(timeout), actor.self(), (Runnable) this::cancel,
+ actor.getContext().system().dispatcher(), actor.self());
}
@Override
public void run() {
- if(canRun) {
+ if (canRun) {
canRun = false;
cancelTimer.cancel();
doRun();