X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;ds=sidebyside;f=opendaylight%2Fmd-sal%2Fsal-distributed-datastore%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fdatastore%2FDataChangeListener.java;h=dc19aa400f66d47fd59134b062f8b04c9bfb6f78;hb=a3cecfd01d0ef8922530924e3ee9684eb03ee5d6;hp=e1f27773732d8d2718c774ffd52928d7afef2941;hpb=057b787289f7b909d7013c22ac73a1c91c860af8;p=controller.git 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 e1f2777373..dc19aa400f 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,9 +9,7 @@ package org.opendaylight.controller.cluster.datastore; 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.cluster.datastore.messages.DataChanged; import org.opendaylight.controller.cluster.datastore.messages.DataChangedReply; @@ -25,15 +23,18 @@ import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; /** * Actor for a DataChangeListener. * - * @Deprecated Replaced by {@link DataTreeChangeListener} + * @deprecated Replaced by {@link DataTreeChangeListener} */ @Deprecated public class DataChangeListener extends AbstractUntypedActor { private final AsyncDataChangeListener> listener; + private final YangInstanceIdentifier registeredPath; private boolean notificationsEnabled = false; - public DataChangeListener(AsyncDataChangeListener> listener) { + public DataChangeListener(AsyncDataChangeListener> listener, + final YangInstanceIdentifier registeredPath) { this.listener = Preconditions.checkNotNull(listener, "listener should not be null"); + this.registeredPath = Preconditions.checkNotNull(registeredPath); } @Override @@ -70,7 +71,7 @@ public class DataChangeListener extends AbstractUntypedActor { try { this.listener.onDataChanged(change); } catch (RuntimeException e) { - LOG.error( String.format( "Error notifying listener %s", this.listener ), e ); + LOG.error(String.format("Error notifying listener %s", this.listener), e); } if (isValidSender(getSender())) { @@ -78,26 +79,8 @@ public class DataChangeListener extends AbstractUntypedActor { } } - public static Props props(final AsyncDataChangeListener> listener) { - return Props.create(new DataChangeListenerCreator(listener)); - } - - private static class DataChangeListenerCreator implements Creator { - private static final long serialVersionUID = 1L; - - @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.") - final AsyncDataChangeListener> listener; - - DataChangeListenerCreator( - AsyncDataChangeListener> listener) { - this.listener = listener; - } - - @Override - public DataChangeListener create() throws Exception { - return new DataChangeListener(listener); - } + public static Props props(final AsyncDataChangeListener> listener, + final YangInstanceIdentifier registeredPath) { + return Props.create(DataChangeListener.class, listener, registeredPath); } }