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%2FEntityOwnerChangeListener.java;h=050ee5d4a57214092abbf07cb736610b91b0e64c;hb=a47dd7a5d21ca68804a6d0e2e3ca765f223c2ef4;hp=d10bc2158ae9824a5f6946d62d055da27895f35e;hpb=ac3065a7d9b6666aa51c1507d9e8143a0cb4db07;p=controller.git diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnerChangeListener.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnerChangeListener.java index d10bc2158a..050ee5d4a5 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnerChangeListener.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/datastore/entityownership/EntityOwnerChangeListener.java @@ -8,10 +8,14 @@ package org.opendaylight.controller.cluster.datastore.entityownership; import static org.opendaylight.controller.cluster.datastore.entityownership.EntityOwnersModel.createEntity; -import com.google.common.base.Objects; import com.google.common.base.Optional; +import com.google.common.base.Preconditions; +import com.google.common.base.Strings; +import com.google.common.base.Verify; import java.util.Collection; -import org.opendaylight.controller.md.sal.common.api.clustering.Entity; +import java.util.Objects; +import org.opendaylight.controller.cluster.access.concepts.MemberName; +import org.opendaylight.mdsal.eos.dom.api.DOMEntity; import org.opendaylight.yangtools.yang.data.api.schema.LeafNode; import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeCandidate; @@ -30,13 +34,13 @@ class EntityOwnerChangeListener extends AbstractEntityOwnerChangeListener { private final String localMemberName; private final EntityOwnershipListenerSupport listenerSupport; - EntityOwnerChangeListener(String localMemberName, EntityOwnershipListenerSupport listenerSupport) { - this.localMemberName = localMemberName; - this.listenerSupport = listenerSupport; + EntityOwnerChangeListener(final MemberName localMemberName, final EntityOwnershipListenerSupport listenerSupport) { + this.localMemberName = Verify.verifyNotNull(localMemberName.getName()); + this.listenerSupport = Preconditions.checkNotNull(listenerSupport); } @Override - public void onDataTreeChanged(Collection changes) { + public void onDataTreeChanged(final Collection changes) { for(DataTreeCandidate change: changes) { DataTreeCandidateNode changeRoot = change.getRootNode(); LeafNode ownerLeaf = (LeafNode) changeRoot.getDataAfter().get(); @@ -53,12 +57,12 @@ class EntityOwnerChangeListener extends AbstractEntityOwnerChangeListener { LOG.debug("{}: New owner: {}, Original owner: {}", logId(), newOwner, origOwner); - if(!Objects.equal(origOwner, newOwner)) { - boolean isOwner = Objects.equal(localMemberName, newOwner); - boolean wasOwner = Objects.equal(localMemberName, origOwner); - boolean hasOwner = newOwner != null && !newOwner.toString().isEmpty(); + if (!Objects.equals(origOwner, newOwner)) { + boolean isOwner = localMemberName.equals(newOwner); + boolean wasOwner = localMemberName.equals(origOwner); + boolean hasOwner = !Strings.isNullOrEmpty(newOwner); - Entity entity = createEntity(change.getRootPath()); + DOMEntity entity = createEntity(change.getRootPath()); LOG.debug("{}: Calling notifyEntityOwnershipListeners: entity: {}, wasOwner: {}, isOwner: {}, hasOwner: {}", logId(), entity, wasOwner, isOwner, hasOwner);