X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fconfig%2Fconfig-manager%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fconfig%2Fmanager%2Fimpl%2Fdependencyresolver%2FDependencyResolverImpl.java;h=065a0f843f501cdb7e2a4230c9cc0b72caf31f3c;hp=87b2e1f752525e5cce27a0b883f8e247a8987e3a;hb=caee336f062eba4909ba53cbaccdde0714236134;hpb=9fb64948564e252018f9b1e13e7cea2c92f991aa diff --git a/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/dependencyresolver/DependencyResolverImpl.java b/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/dependencyresolver/DependencyResolverImpl.java index 87b2e1f752..065a0f843f 100644 --- a/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/dependencyresolver/DependencyResolverImpl.java +++ b/opendaylight/config/config-manager/src/main/java/org/opendaylight/controller/config/manager/impl/dependencyresolver/DependencyResolverImpl.java @@ -7,15 +7,6 @@ */ package org.opendaylight.controller.config.manager.impl.dependencyresolver; -import static java.lang.String.format; - -import java.util.HashSet; -import java.util.LinkedHashSet; -import java.util.Set; - -import javax.annotation.concurrent.GuardedBy; -import javax.management.ObjectName; - import org.opendaylight.controller.config.api.DependencyResolver; import org.opendaylight.controller.config.api.JmxAttribute; import org.opendaylight.controller.config.api.JmxAttributeValidationException; @@ -25,7 +16,14 @@ import org.opendaylight.controller.config.api.jmx.ObjectNameUtil; import org.opendaylight.controller.config.manager.impl.TransactionStatus; import org.opendaylight.controller.config.spi.Module; import org.opendaylight.controller.config.spi.ModuleFactory; -import org.opendaylight.protocol.concepts.NamedObject; + +import javax.annotation.concurrent.GuardedBy; +import javax.management.ObjectName; +import java.util.HashSet; +import java.util.LinkedHashSet; +import java.util.Set; + +import static java.lang.String.format; /** * Protect {@link org.opendaylight.controller.config.spi.Module#getInstance()} @@ -33,7 +31,7 @@ import org.opendaylight.protocol.concepts.NamedObject; * during validation. Tracks dependencies for ordering purposes. */ final class DependencyResolverImpl implements DependencyResolver, - NamedObject, Comparable { + Comparable { private final ModulesHolder modulesHolder; private final ModuleIdentifier name; private final TransactionStatus transactionStatus; @@ -47,11 +45,6 @@ final class DependencyResolverImpl implements DependencyResolver, this.modulesHolder = modulesHolder; } - @Override - public ModuleIdentifier getName() { - return name; - } - /** * {@inheritDoc} */ @@ -105,14 +98,6 @@ final class DependencyResolverImpl implements DependencyResolver, } } - @Override - public void validateDependency( - Class expectedServiceInterface, - ObjectName objectName, String attributeNameForErrorReporting) { - validateDependency(expectedServiceInterface, objectName, - new JmxAttribute(attributeNameForErrorReporting)); - } - /** * {@inheritDoc} */ @@ -155,13 +140,6 @@ final class DependencyResolverImpl implements DependencyResolver, } } - @Deprecated - @Override - public T resolveInstance(Class expectedType, ObjectName objectName) { - return resolveInstance(expectedType, objectName, new JmxAttribute( - "unknown attribute")); - } - @Override public int compareTo(DependencyResolverImpl o) { transactionStatus.checkCommitted(); @@ -192,7 +170,7 @@ final class DependencyResolverImpl implements DependencyResolver, int maxDepth = 0; LinkedHashSet chainForDetectingCycles2 = new LinkedHashSet<>( chainForDetectingCycles); - chainForDetectingCycles2.add(impl.getName()); + chainForDetectingCycles2.add(impl.getIdentifier()); for (ModuleIdentifier dependencyName : impl.dependencies) { DependencyResolverImpl dependentDRI = manager .getOrCreate(dependencyName); @@ -216,4 +194,9 @@ final class DependencyResolverImpl implements DependencyResolver, impl.maxDependencyDepth = maxDepth; return maxDepth; } + + @Override + public ModuleIdentifier getIdentifier() { + return name; + } }