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=fbffd5b11a207a0e8906f08b30bd257a470ffe58;hb=a3cecfd01d0ef8922530924e3ee9684eb03ee5d6;hp=8c6b04cbf3c1eeeaccf181dfc0f9a612685a8123;hpb=1f3f61ea49191bb0ada2d4de831a10f0a38a104d;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 8c6b04cbf3..fbffd5b11a 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 @@ -10,11 +10,10 @@ package org.opendaylight.controller.cluster.datastore.entityownership; 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.controller.md.sal.common.api.clustering.EntityOwnershipChange; -import org.opendaylight.controller.md.sal.common.api.clustering.EntityOwnershipListener; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import org.opendaylight.mdsal.eos.dom.api.DOMEntityOwnershipChange; +import org.opendaylight.mdsal.eos.dom.api.DOMEntityOwnershipListener; /** * An actor which is responsible for notifying an EntityOwnershipListener of changes. @@ -22,22 +21,23 @@ import org.slf4j.LoggerFactory; * @author Thomas Pantelis */ class EntityOwnershipListenerActor extends AbstractUntypedActor { - private static final Logger LOG = LoggerFactory.getLogger(EntityOwnershipListenerActor.class); + private final DOMEntityOwnershipListener listener; - private final EntityOwnershipListener listener; - - private EntityOwnershipListenerActor(EntityOwnershipListener listener) { + private EntityOwnershipListenerActor(DOMEntityOwnershipListener listener) { this.listener = listener; } @Override protected void handleReceive(Object message) { - if(message instanceof EntityOwnershipChange) { - onEntityOwnershipChanged((EntityOwnershipChange)message); + if (message instanceof DOMEntityOwnershipChange) { + onEntityOwnershipChanged((DOMEntityOwnershipChange)message); + } else { + unknownMessage(message); } } - private void onEntityOwnershipChanged(EntityOwnershipChange change) { + @SuppressWarnings("checkstyle:IllegalCatch") + private void onEntityOwnershipChanged(DOMEntityOwnershipChange change) { LOG.debug("Notifying EntityOwnershipListener {}: {}", listener, change); try { @@ -47,16 +47,18 @@ class EntityOwnershipListenerActor extends AbstractUntypedActor { } } - static Props props(EntityOwnershipListener listener) { + static Props props(DOMEntityOwnershipListener listener) { return Props.create(new EntityOwnershipListenerCreator(listener)); } private static final class EntityOwnershipListenerCreator implements Creator { private static final long serialVersionUID = 1L; - private final EntityOwnershipListener listener; + @SuppressFBWarnings(value = "SE_BAD_FIELD", justification = "This field is not Serializable but we don't " + + "create remote instances of this actor and thus don't need it to be Serializable.") + private final DOMEntityOwnershipListener listener; - EntityOwnershipListenerCreator(EntityOwnershipListener listener) { + EntityOwnershipListenerCreator(DOMEntityOwnershipListener listener) { this.listener = Preconditions.checkNotNull(listener); }