X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fconfig%2Fconfig-api%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fconfig%2Fapi%2Fjmx%2FObjectNameUtil.java;h=3b7858c4771c3419f2958415a2e852865f27d851;hp=c23a0cbf69850f364113a93579e79d9dc952ae5c;hb=f43b01b81319959b1907e3e04537f5169e7f33d8;hpb=0eba94d9411ea40945ddc8c732640c0cc004599f diff --git a/opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/jmx/ObjectNameUtil.java b/opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/jmx/ObjectNameUtil.java index c23a0cbf69..3b7858c477 100644 --- a/opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/jmx/ObjectNameUtil.java +++ b/opendaylight/config/config-api/src/main/java/org/opendaylight/controller/config/api/jmx/ObjectNameUtil.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2013 Cisco Systems, Inc. and others. All rights reserved. + * Copyright (c) 2013, 2017 Cisco Systems, Inc. and others. All rights reserved. * * This program and the accompanying materials are made available under the * terms of the Eclipse Public License v1.0 which accompanies this distribution, @@ -7,12 +7,6 @@ */ package org.opendaylight.controller.config.api.jmx; -import org.opendaylight.controller.config.api.ModuleIdentifier; -import org.opendaylight.controller.config.api.jmx.constants.ConfigRegistryConstants; - -import javax.annotation.concurrent.ThreadSafe; -import javax.management.MalformedObjectNameException; -import javax.management.ObjectName; import java.util.Arrays; import java.util.HashMap; import java.util.HashSet; @@ -20,6 +14,11 @@ import java.util.Hashtable; import java.util.Map; import java.util.Map.Entry; import java.util.Set; +import javax.annotation.concurrent.ThreadSafe; +import javax.management.MalformedObjectNameException; +import javax.management.ObjectName; +import org.opendaylight.controller.config.api.ModuleIdentifier; +import org.opendaylight.controller.config.api.jmx.constants.ConfigRegistryConstants; /** * Provides ObjectName creation. Each created ObjectName consists of domain that @@ -28,9 +27,7 @@ import java.util.Set; * {@link #TRANSACTION_NAME_KEY} property set. */ @ThreadSafe -public class ObjectNameUtil { - private ObjectNameUtil() { - } +public final class ObjectNameUtil { public static final String ON_DOMAIN = ConfigRegistryConstants.ON_DOMAIN; public static final String MODULE_FACTORY_NAME_KEY = "moduleFactoryName"; @@ -46,90 +43,94 @@ public class ObjectNameUtil { private static final String REPLACED_QUOTATION_MARK = "\\?"; public static final String ON_WILDCARD = "*"; - public static ObjectName createON(String on) { + private ObjectNameUtil() { + } + + public static ObjectName createON(final String on) { try { return new ObjectName(on); - } catch (MalformedObjectNameException e) { + } catch (final MalformedObjectNameException e) { throw new IllegalArgumentException(e); } } - public static ObjectName createONWithDomainAndType(String type) { - return ConfigRegistryConstants.createONWithDomainAndType(type); - } - - public static ObjectName createON(String name, String key, String value) { + public static ObjectName createON(final String name, final String key, final String value) { return ConfigRegistryConstants.createON(name, key, value); } - public static ObjectName createON(String domain, Map attribs) { + public static ObjectName createON(final String domain, final Map attribs) { Hashtable table = new Hashtable<>(attribs); try { return new ObjectName(domain, table); - } catch (MalformedObjectNameException e) { + } catch (final MalformedObjectNameException e) { throw new IllegalArgumentException(e); } } - public static ObjectName createTransactionControllerON( - String transactionName) { + public static ObjectName createONWithDomainAndType(final String type) { + return ConfigRegistryConstants.createONWithDomainAndType(type); + } + + public static ObjectName createTransactionControllerON(final String transactionName) { Map onParams = new HashMap<>(); onParams.put(TRANSACTION_NAME_KEY, transactionName); onParams.put(TYPE_KEY, TYPE_CONFIG_TRANSACTION); return createON(ON_DOMAIN, onParams); } - public static ObjectName createTransactionModuleON(String transactionName, - ModuleIdentifier moduleIdentifier) { - return createTransactionModuleON(transactionName, - moduleIdentifier.getFactoryName(), + public static ObjectName createTransactionModuleON(final String transactionName, + final ModuleIdentifier moduleIdentifier) { + return createTransactionModuleON(transactionName, moduleIdentifier.getFactoryName(), moduleIdentifier.getInstanceName()); } - public static ObjectName createTransactionModuleON(String transactionName, - String moduleName, String instanceName) { + public static ObjectName createTransactionModuleON(final String transactionName, final String moduleName, + final String instanceName) { Map onParams = createModuleMap(moduleName, instanceName); onParams.put(TRANSACTION_NAME_KEY, transactionName); return createON(ON_DOMAIN, onParams); } - public static ObjectName createTransactionModuleON(String transactionName, - ObjectName on) { - return createTransactionModuleON(transactionName, getFactoryName(on), - getInstanceName(on)); + public static ObjectName createTransactionModuleON(final String transactionName, final ObjectName on) { + return createTransactionModuleON(transactionName, getFactoryName(on), getInstanceName(on)); } - public static ObjectName createReadOnlyModuleON( - ModuleIdentifier moduleIdentifier) { - return createReadOnlyModuleON(moduleIdentifier.getFactoryName(), - moduleIdentifier.getInstanceName()); + public static ObjectName createReadOnlyModuleON(final ModuleIdentifier moduleIdentifier) { + return createReadOnlyModuleON(moduleIdentifier.getFactoryName(), moduleIdentifier.getInstanceName()); + } + + public static ObjectName createReadOnlyModuleON(final String moduleName, final String instanceName) { + Map onParams = createModuleMap(moduleName, instanceName); + return createON(ON_DOMAIN, onParams); } - public static ObjectName createReadOnlyServiceON(String serviceQName, String refName) { + public static ObjectName createReadOnlyServiceON(final String serviceQName, final String refName) { Map onParams = createServiceMap(serviceQName, refName); return createON(ON_DOMAIN, onParams); } - public static ObjectName createTransactionServiceON(String transactionName, String serviceQName, String refName) { + public static ObjectName createTransactionServiceON(final String transactionName, final String serviceQName, + final String refName) { Map onParams = createServiceON(transactionName, serviceQName, refName); return createON(ON_DOMAIN, onParams); } - public static String getServiceQName(ObjectName objectName) { + public static String getServiceQName(final ObjectName objectName) { checkType(objectName, TYPE_SERVICE_REFERENCE); String quoted = objectName.getKeyProperty(SERVICE_QNAME_KEY); return unquoteAndUnescape(objectName, quoted); } - // ObjectName supports quotation and ignores tokens like =, but fails to ignore ? sign. + // ObjectName supports quotation and ignores tokens like =, but fails to ignore + // ? sign. // It must be replaced with another character that hopefully does not collide // with actual value. - private static String unquoteAndUnescape(ObjectName objectName, String quoted) { + private static String unquoteAndUnescape(final ObjectName objectName, final String quoted) { if (quoted == null) { throw new IllegalArgumentException("Cannot find " + SERVICE_QNAME_KEY + " in " + objectName); } - if (quoted.startsWith("\"") == false || quoted.endsWith("\"") == false) { + if (!quoted.startsWith("\"") || !quoted.endsWith("\"")) { throw new IllegalArgumentException("Quotes not found in " + objectName); } String substring = quoted.substring(1); @@ -138,24 +139,23 @@ public class ObjectNameUtil { return substring; } - private static String quoteAndEscapeValue(String serviceQName) { + private static String quoteAndEscapeValue(final String serviceQName) { return "\"" + serviceQName.replace("?", REPLACED_QUOTATION_MARK) + "\""; } - public static String getReferenceName(ObjectName objectName) { + public static String getReferenceName(final ObjectName objectName) { checkType(objectName, TYPE_SERVICE_REFERENCE); return objectName.getKeyProperty(REF_NAME_KEY); } - private static Map createServiceON(String transactionName, String serviceQName, - String refName) { + private static Map createServiceON(final String transactionName, final String serviceQName, + final String refName) { Map result = new HashMap<>(createServiceMap(serviceQName, refName)); result.put(TRANSACTION_NAME_KEY, transactionName); return result; } - private static Map createServiceMap(String serviceQName, - String refName) { + private static Map createServiceMap(final String serviceQName, final String refName) { Map onParams = new HashMap<>(); onParams.put(TYPE_KEY, TYPE_SERVICE_REFERENCE); onParams.put(SERVICE_QNAME_KEY, quoteAndEscapeValue(serviceQName)); @@ -163,15 +163,7 @@ public class ObjectNameUtil { return onParams; } - - public static ObjectName createReadOnlyModuleON(String moduleName, - String instanceName) { - Map onParams = createModuleMap(moduleName, instanceName); - return createON(ON_DOMAIN, onParams); - } - - private static Map createModuleMap(String moduleName, - String instanceName) { + private static Map createModuleMap(final String moduleName, final String instanceName) { Map onParams = new HashMap<>(); onParams.put(TYPE_KEY, TYPE_MODULE); onParams.put(MODULE_FACTORY_NAME_KEY, moduleName); @@ -179,32 +171,30 @@ public class ObjectNameUtil { return onParams; } - public static String getFactoryName(ObjectName objectName) { + public static String getFactoryName(final ObjectName objectName) { checkTypeOneOf(objectName, TYPE_MODULE, TYPE_RUNTIME_BEAN); return objectName.getKeyProperty(MODULE_FACTORY_NAME_KEY); } - public static String getInstanceName(ObjectName objectName) { + public static String getInstanceName(final ObjectName objectName) { checkTypeOneOf(objectName, TYPE_MODULE, TYPE_RUNTIME_BEAN); return objectName.getKeyProperty(INSTANCE_NAME_KEY); } - public static String getTransactionName(ObjectName objectName) { + public static String getTransactionName(final ObjectName objectName) { return objectName.getKeyProperty(TRANSACTION_NAME_KEY); } /** * Sanitize on: keep only mandatory attributes of module + metadata. */ - public static ObjectName withoutTransactionName(ObjectName inputON) { + public static ObjectName withoutTransactionName(final ObjectName inputON) { checkTypeOneOf(inputON, TYPE_MODULE, TYPE_SERVICE_REFERENCE); if (getTransactionName(inputON) == null) { - throw new IllegalArgumentException( - "Expected ObjectName with transaction:" + inputON); + throw new IllegalArgumentException("Expected ObjectName with transaction:" + inputON); } - if (ON_DOMAIN.equals(inputON.getDomain()) == false) { - throw new IllegalArgumentException("Expected different domain: " - + inputON); + if (!ON_DOMAIN.equals(inputON.getDomain())) { + throw new IllegalArgumentException("Expected different domain: " + inputON); } Map outputProperties; if (inputON.getKeyProperty(TYPE_KEY).equals(TYPE_MODULE)) { @@ -225,24 +215,21 @@ public class ObjectNameUtil { return createON(ON_DOMAIN, outputProperties); } - public static ObjectName withTransactionName(ObjectName inputON, String transactionName) { + public static ObjectName withTransactionName(final ObjectName inputON, final String transactionName) { Map additionalProperties = getAdditionalProperties(inputON); additionalProperties.put(TRANSACTION_NAME_KEY, transactionName); return createON(inputON.getDomain(), additionalProperties); } - private static void assertDoesNotContain( - Map additionalProperties, String key) { + private static void assertDoesNotContain(final Map additionalProperties, final String key) { if (additionalProperties.containsKey(key)) { - throw new IllegalArgumentException( - "Map 'additionalProperties' cannot overwrite attribute " - + key); + throw new IllegalArgumentException("Map 'additionalProperties' cannot overwrite attribute " + key); } } - public static ObjectName createRuntimeBeanName(String moduleName, - String instanceName, Map additionalProperties) { + public static ObjectName createRuntimeBeanName(final String moduleName, final String instanceName, + final Map additionalProperties) { // check that there is no overwriting of default attributes assertDoesNotContain(additionalProperties, MODULE_FACTORY_NAME_KEY); assertDoesNotContain(additionalProperties, INSTANCE_NAME_KEY); @@ -255,23 +242,22 @@ public class ObjectNameUtil { return createON(ON_DOMAIN, map); } - private static Set blacklist = new HashSet<>(Arrays.asList( - MODULE_FACTORY_NAME_KEY, INSTANCE_NAME_KEY, TYPE_KEY)); + private static Set blacklist = new HashSet<>( + Arrays.asList(MODULE_FACTORY_NAME_KEY, INSTANCE_NAME_KEY, TYPE_KEY)); - public static Map getAdditionalPropertiesOfRuntimeBeanName( - ObjectName on) { + public static Map getAdditionalPropertiesOfRuntimeBeanName(final ObjectName on) { checkType(on, TYPE_RUNTIME_BEAN); Map allProperties = getAdditionalProperties(on); Map result = new HashMap<>(); for (Entry entry : allProperties.entrySet()) { - if (blacklist.contains(entry.getKey()) == false) { + if (!blacklist.contains(entry.getKey())) { result.put(entry.getKey(), entry.getValue()); } } return result; } - public static Map getAdditionalProperties(ObjectName on) { + public static Map getAdditionalProperties(final ObjectName on) { Map keyPropertyList = on.getKeyPropertyList(); Map result = new HashMap<>(); for (Entry entry : keyPropertyList.entrySet()) { @@ -280,90 +266,75 @@ public class ObjectNameUtil { return result; } - public static void checkDomain(ObjectName objectName) { + public static void checkDomain(final ObjectName objectName) { if (!ON_DOMAIN.equals(objectName.getDomain())) { throw new IllegalArgumentException("Wrong domain " + objectName); } } - public static void checkType(ObjectName objectName, String type) { + public static void checkType(final ObjectName objectName, final String type) { if (!type.equals(objectName.getKeyProperty(TYPE_KEY))) { - throw new IllegalArgumentException("Wrong type, expected '" + type - + "', got " + objectName); + throw new IllegalArgumentException("Wrong type, expected '" + type + "', got " + objectName); } } - public static void checkTypeOneOf(ObjectName objectName, String... types) { + public static void checkTypeOneOf(final ObjectName objectName, final String... types) { for (String type : types) { if (type.equals(objectName.getKeyProperty(TYPE_KEY))) { return; } } - throw new IllegalArgumentException("Wrong type, expected one of " + Arrays.asList(types) - + ", got " + objectName); + throw new IllegalArgumentException( + "Wrong type, expected one of " + Arrays.asList(types) + ", got " + objectName); } - public static ObjectName createModulePattern(String moduleName, - String instanceName) { + public static ObjectName createModulePattern(final String moduleName, final String instanceName) { String finalModuleName = moduleName == null ? ON_WILDCARD : moduleName; String finalInstanceName = instanceName == null ? ON_WILDCARD : instanceName; // do not return object names containing transaction name - ObjectName namePattern = ObjectNameUtil - .createON(ObjectNameUtil.ON_DOMAIN + ":" - + ObjectNameUtil.TYPE_KEY + "=" - + ObjectNameUtil.TYPE_MODULE + "," - + ObjectNameUtil.MODULE_FACTORY_NAME_KEY + "=" - + finalModuleName + "," + "" - + ObjectNameUtil.INSTANCE_NAME_KEY + "=" + finalInstanceName); + ObjectName namePattern = ObjectNameUtil.createON(ObjectNameUtil.ON_DOMAIN + ":" + ObjectNameUtil.TYPE_KEY + "=" + + ObjectNameUtil.TYPE_MODULE + "," + ObjectNameUtil.MODULE_FACTORY_NAME_KEY + "=" + finalModuleName + + "," + "" + ObjectNameUtil.INSTANCE_NAME_KEY + "=" + finalInstanceName); return namePattern; } - public static ObjectName createModulePattern(String ifcName, - String instanceName, String transactionName) { + public static ObjectName createModulePattern(final String ifcName, final String instanceName, + final String transactionName) { String finalIfcName = ifcName == null ? ON_WILDCARD : ifcName; String finalInstanceName = instanceName == null ? ON_WILDCARD : instanceName; String finalTransactionName = transactionName == null ? ON_WILDCARD : transactionName; - return ObjectNameUtil.createON(ObjectNameUtil.ON_DOMAIN - + ":type=Module," + ObjectNameUtil.MODULE_FACTORY_NAME_KEY - + "=" + finalIfcName + "," + ObjectNameUtil.INSTANCE_NAME_KEY + "=" - + finalInstanceName + "," + ObjectNameUtil.TRANSACTION_NAME_KEY - + "=" + finalTransactionName); + return ObjectNameUtil.createON(ObjectNameUtil.ON_DOMAIN + ":type=Module," + + ObjectNameUtil.MODULE_FACTORY_NAME_KEY + "=" + finalIfcName + "," + ObjectNameUtil.INSTANCE_NAME_KEY + + "=" + finalInstanceName + "," + ObjectNameUtil.TRANSACTION_NAME_KEY + "=" + finalTransactionName); } - public static ObjectName createRuntimeBeanPattern(String moduleName, - String instanceName) { + public static ObjectName createRuntimeBeanPattern(final String moduleName, final String instanceName) { String finalModuleName = moduleName == null ? ON_WILDCARD : moduleName; String finalInstanceName = instanceName == null ? ON_WILDCARD : instanceName; - return ObjectNameUtil.createON(ObjectNameUtil.ON_DOMAIN + ":" - + ObjectNameUtil.TYPE_KEY + "=" - + ObjectNameUtil.TYPE_RUNTIME_BEAN + "," - + ObjectNameUtil.MODULE_FACTORY_NAME_KEY + "=" + finalModuleName - + "," + ObjectNameUtil.INSTANCE_NAME_KEY + "=" + finalInstanceName - + ",*"); + return ObjectNameUtil.createON(ObjectNameUtil.ON_DOMAIN + ":" + ObjectNameUtil.TYPE_KEY + "=" + + ObjectNameUtil.TYPE_RUNTIME_BEAN + "," + ObjectNameUtil.MODULE_FACTORY_NAME_KEY + "=" + + finalModuleName + "," + ObjectNameUtil.INSTANCE_NAME_KEY + "=" + finalInstanceName + ",*"); } - public static ModuleIdentifier fromON(ObjectName objectName, - String expectedType) { + public static ModuleIdentifier fromON(final ObjectName objectName, final String expectedType) { checkType(objectName, expectedType); String factoryName = getFactoryName(objectName); if (factoryName == null) { - throw new IllegalArgumentException( - "ObjectName does not contain module name"); + throw new IllegalArgumentException("ObjectName does not contain module name"); } String instanceName = getInstanceName(objectName); if (instanceName == null) { - throw new IllegalArgumentException( - "ObjectName does not contain instance name"); + throw new IllegalArgumentException("ObjectName does not contain instance name"); } return new ModuleIdentifier(factoryName, instanceName); } - public static boolean isServiceReference(ObjectName objectName) { + public static boolean isServiceReference(final ObjectName objectName) { return TYPE_SERVICE_REFERENCE.equals(objectName.getKeyProperty(TYPE_KEY)); } }