X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2Fentityownership%2FEntityOwnershipListenerActor.java;h=037f6b66f7b27e7efb97ef40f377ce98036438d6;hb=refs%2Fchanges%2F71%2F85371%2F3;hp=fbffd5b11a207a0e8906f08b30bd257a470ffe58;hpb=057b787289f7b909d7013c22ac73a1c91c860af8;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnershipListenerActor.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnershipListenerActor.java index fbffd5b11a..037f6b66f7 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnershipListenerActor.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnershipListenerActor.java @@ -7,9 +7,10 @@ */ package org.opendaylight.controller.cluster.datastore.entityownership; +import static java.util.Objects.requireNonNull; + import akka.actor.Props; import akka.japi.Creator; -import com.google.common.base.Preconditions; import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; import org.opendaylight.controller.cluster.common.actor.AbstractUntypedActor; import org.opendaylight.mdsal.eos.dom.api.DOMEntityOwnershipChange; @@ -20,15 +21,15 @@ import org.opendaylight.mdsal.eos.dom.api.DOMEntityOwnershipListener; * * @author Thomas Pantelis */ -class EntityOwnershipListenerActor extends AbstractUntypedActor { +final class EntityOwnershipListenerActor extends AbstractUntypedActor { private final DOMEntityOwnershipListener listener; - private EntityOwnershipListenerActor(DOMEntityOwnershipListener listener) { + private EntityOwnershipListenerActor(final DOMEntityOwnershipListener listener) { this.listener = listener; } @Override - protected void handleReceive(Object message) { + protected void handleReceive(final Object message) { if (message instanceof DOMEntityOwnershipChange) { onEntityOwnershipChanged((DOMEntityOwnershipChange)message); } else { @@ -37,7 +38,7 @@ class EntityOwnershipListenerActor extends AbstractUntypedActor { } @SuppressWarnings("checkstyle:IllegalCatch") - private void onEntityOwnershipChanged(DOMEntityOwnershipChange change) { + private void onEntityOwnershipChanged(final DOMEntityOwnershipChange change) { LOG.debug("Notifying EntityOwnershipListener {}: {}", listener, change); try { @@ -47,8 +48,8 @@ class EntityOwnershipListenerActor extends AbstractUntypedActor { } } - static Props props(DOMEntityOwnershipListener listener) { - return Props.create(new EntityOwnershipListenerCreator(listener)); + static Props props(final DOMEntityOwnershipListener listener) { + return Props.create(EntityOwnershipListenerActor.class, new EntityOwnershipListenerCreator(listener)); } private static final class EntityOwnershipListenerCreator implements Creator { @@ -58,8 +59,8 @@ class EntityOwnershipListenerActor extends AbstractUntypedActor { + "create remote instances of this actor and thus don't need it to be Serializable.") private final DOMEntityOwnershipListener listener; - EntityOwnershipListenerCreator(DOMEntityOwnershipListener listener) { - this.listener = Preconditions.checkNotNull(listener); + EntityOwnershipListenerCreator(final DOMEntityOwnershipListener listener) { + this.listener = requireNonNull(listener); } @Override