diff options
Diffstat (limited to 'policy-management/src/main/java/org/onap/policy/drools/persistence/SystemPersistence.java')
-rw-r--r-- | policy-management/src/main/java/org/onap/policy/drools/persistence/SystemPersistence.java | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/policy-management/src/main/java/org/onap/policy/drools/persistence/SystemPersistence.java b/policy-management/src/main/java/org/onap/policy/drools/persistence/SystemPersistence.java index 52aac744..bdda8e84 100644 --- a/policy-management/src/main/java/org/onap/policy/drools/persistence/SystemPersistence.java +++ b/policy-management/src/main/java/org/onap/policy/drools/persistence/SystemPersistence.java @@ -69,11 +69,11 @@ public interface SystemPersistence { * * @param controllerName the controller name * @param configuration object containing the configuration + * * @return true if storage is succesful, false otherwise * @throws IllegalArgumentException if the configuration cannot be handled by the persistence manager */ - public boolean storeController(String controllerName, Object configuration) - throws IllegalArgumentException; + public boolean storeController(String controllerName, Object configuration); /** * delete controller configuration @@ -88,19 +88,20 @@ public interface SystemPersistence { * * @param controllerName controller name * @return properties for this controller + * * @throws IllegalArgumentException if the controller name does not lead to a properties configuration */ - public Properties getControllerProperties(String controllerName) - throws IllegalArgumentException; + public Properties getControllerProperties(String controllerName); /** * get properties by name * * @param name * @return properties + * * @throws IllegalArgumentException if the name does not lead to a properties configuration */ - public Properties getProperties(String name) throws IllegalArgumentException; + public Properties getProperties(String name); /** * Persistence Manager. For now it is a file-based properties management, @@ -196,6 +197,7 @@ class SystemPropertiesPersistence implements SystemPersistence { /** * deletes properties-based controller configuration * @param controllerName the controller name + * * @return true if the properties has been deleted from disk, false otherwise */ @Override @@ -223,12 +225,12 @@ class SystemPropertiesPersistence implements SystemPersistence { } @Override - public Properties getControllerProperties(String controllerName) throws IllegalArgumentException { + public Properties getControllerProperties(String controllerName) { return this.getProperties(controllerName + CONTROLLER_SUFFIX_IDENTIFIER); } @Override - public Properties getProperties(String name) throws IllegalArgumentException { + public Properties getProperties(String name) { Path propertiesPath = Paths.get(CONFIG_DIR_NAME, name + ".properties"); @@ -242,7 +244,7 @@ class SystemPropertiesPersistence implements SystemPersistence { logger.warn("{}: can't read properties @ {}", name, propertiesPath); throw new IllegalArgumentException("can't read properties for " + name + " @ " + - propertiesPath); + propertiesPath, e); } } } |