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%2FDataChangeListener.java;h=1bc835f1e3c6883196d4a9a6b1f334bb16f9dca7;hp=ba09d0402530921667915eae11103302cfcfb2ea;hb=37f0504d391efd8b7d61403759fcc22a6dd3a093;hpb=e2df8aa67238c153f1038eb45f7799442861985b diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DataChangeListener.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DataChangeListener.java index ba09d04025..1bc835f1e3 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DataChangeListener.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/DataChangeListener.java @@ -9,22 +9,90 @@ package org.opendaylight.controller.cluster.datastore; import akka.actor.Props; -import akka.actor.UntypedActor; import akka.japi.Creator; +import com.google.common.base.Preconditions; +import org.opendaylight.controller.cluster.common.actor.AbstractUntypedActor; +import org.opendaylight.controller.cluster.datastore.messages.DataChanged; +import org.opendaylight.controller.cluster.datastore.messages.DataChangedReply; +import org.opendaylight.controller.cluster.datastore.messages.EnableNotification; +import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeEvent; +import org.opendaylight.controller.md.sal.common.api.data.AsyncDataChangeListener; +import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; +import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; -public class DataChangeListener extends UntypedActor { - @Override public void onReceive(Object message) throws Exception { - throw new UnsupportedOperationException("onReceive"); +public class DataChangeListener extends AbstractUntypedActor { + private static final Logger LOG = LoggerFactory.getLogger(DataChangeListener.class); + + private final AsyncDataChangeListener> listener; + private boolean notificationsEnabled = false; + + public DataChangeListener(AsyncDataChangeListener> listener) { + this.listener = Preconditions.checkNotNull(listener, "listener should not be null"); + } + + @Override + public void handleReceive(Object message) throws Exception { + if(message instanceof DataChanged){ + dataChanged(message); + } else if(message instanceof EnableNotification){ + enableNotification((EnableNotification) message); + } + } + + private void enableNotification(EnableNotification message) { + notificationsEnabled = message.isEnabled(); + LOG.debug("{} notifications for listener {}", (notificationsEnabled ? "Enabled" : "Disabled"), + listener); + } + + private void dataChanged(Object message) { + + // Do nothing if notifications are not enabled + if(!notificationsEnabled) { + LOG.debug("Notifications not enabled for listener {} - dropping change notification", + listener); + return; + } + + DataChanged reply = (DataChanged) message; + AsyncDataChangeEvent> change = reply.getChange(); + + LOG.debug("Sending change notification {} to listener {}", change, listener); + + try { + this.listener.onDataChanged(change); + } catch (RuntimeException e) { + LOG.error( String.format( "Error notifying listener %s", this.listener ), e ); + } + + // It seems the sender is never null but it doesn't hurt to check. If the caller passes in + // a null sender (ActorRef.noSender()), akka translates that to the deadLetters actor. + if(getSender() != null && !getContext().system().deadLetters().equals(getSender())) { + getSender().tell(DataChangedReply.INSTANCE, getSelf()); + } + } + + public static Props props(final AsyncDataChangeListener> listener) { + return Props.create(new DataChangeListenerCreator(listener)); } - public static Props props() { - return Props.create(new Creator() { - @Override - public DataChangeListener create() throws Exception { - return new DataChangeListener(); - } + private static class DataChangeListenerCreator implements Creator { + private static final long serialVersionUID = 1L; + + final AsyncDataChangeListener> listener; - }); + DataChangeListenerCreator( + AsyncDataChangeListener> listener) { + this.listener = listener; + } + @Override + public DataChangeListener create() throws Exception { + return new DataChangeListener(listener); + } } }