refactoring into the settings subprojects, each content/datastructure

content-api/src/main/java/module-info.java

@@ -1,8 +1,3 @@
-import org.xbib.content.SettingsBuilder;
-
 module org.xbib.content.api {
     exports org.xbib.content;
-    requires transitive org.xbib.datastructures.api;
-    requires java.sql;
-    uses SettingsBuilder;
 }

content-config/build.gradle

@@ -1,5 +1,5 @@
 dependencies {
-    api project(':content-settings-datastructures')
-    testImplementation project(':content-settings-datastructures-json')
-    testImplementation project(':content-settings-datastructures-yaml')
+    api project(':settings-datastructures')
+    testImplementation project(':settings-datastructures-json')
+    testImplementation project(':settings-datastructures-yaml')
 }

content-config/src/main/java/module-info.java

@@ -1,11 +1,11 @@
 import org.xbib.content.config.ConfigLogger;
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.SettingsLoader;
 
 module org.xbib.content.config {
     exports org.xbib.content.config;
     uses ConfigLogger;
     uses SettingsLoader;
     provides ConfigLogger with org.xbib.content.config.SystemConfigLogger;
-    requires org.xbib.content.api;
-    requires transitive org.xbib.content.settings.datastructures;
+    requires org.xbib.settings.api;
+    requires transitive org.xbib.settings.datastructures;
 }

content-config/src/main/java/org/xbib/content/config/ConfigLoader.java

@@ -1,9 +1,9 @@
 package org.xbib.content.config;
 
-import org.xbib.content.Settings;
-import org.xbib.content.SettingsBuilder;
-import org.xbib.content.SettingsLoader;
-import org.xbib.content.settings.datastructures.SettingsLoaderService;
+import org.xbib.settings.Settings;
+import org.xbib.settings.SettingsBuilder;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.SettingsLoaderService;
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStream;
@@ -14,6 +14,7 @@ import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
+import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
@@ -137,6 +138,15 @@ public class ConfigLoader {
                 }
             }
         }
+        if (!params.jdbcLookups.isEmpty()) {
+            for (ConfigParams.JdbcLookup jdbcLookup : params.jdbcLookups) {
+                try {
+                    settings.fromJdbc(jdbcLookup.connection, jdbcLookup.statement, jdbcLookup.params);
+                } catch (SQLException sqlException) {
+                    throw new ConfigException(sqlException);
+                }
+            }
+        }
         if (params.includeAll) {
             return overrideFromProperties(params, settings);
         }
@@ -263,15 +273,15 @@ public class ConfigLoader {
         return null;
     }
 
-    private SettingsBuilder overrideFromProperties(ConfigParams params,
-                                                   SettingsBuilder settings) {
-        for (String key : settings.map().keySet()) {
-            String value = System.getProperty(params.directoryName != null ? params.directoryName + '.' + key : key);
-            if (value != null) {
-                settings.put(key, value);
-            }
+    private SettingsBuilder overrideFromProperties(ConfigParams params, SettingsBuilder settingsBuilder) {
+        if (params.withSystemPropertiesOverride) {
+            settingsBuilder.map(e -> {
+                String key = e.getKey();
+                String value = System.getProperty(params.directoryName != null ? params.directoryName + '.' + key : key);
+                return value != null ? Map.entry(key, value) : Map.entry(key, e.getValue());
+            });
         }
-        return settings;
+        return settingsBuilder;
     }
 
     private List<String> createListOfLocations(ConfigParams params,

content-config/src/main/java/org/xbib/content/config/ConfigParams.java

@@ -1,8 +1,9 @@
 package org.xbib.content.config;
 
-import org.xbib.content.settings.datastructures.DatastructureSettings;
+import org.xbib.settings.datastructures.DatastructureSettings;
 import java.io.IOException;
 import java.io.Reader;
+import java.sql.Connection;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Comparator;
@@ -21,10 +22,14 @@ public class ConfigParams implements Comparable<ConfigParams> {
 
     boolean withStdin = false;
 
+    boolean withSystemPropertiesOverride = false;
+
     List<ClassLoader> classLoaders = null;
 
     final List<SuffixedReader> reader = new ArrayList<>();
 
+    final List<JdbcLookup> jdbcLookups = new ArrayList<>();
+
     final List<DatastructureSettings> settings = new ArrayList<>();
 
     List<String> args = null;
@@ -48,6 +53,11 @@ public class ConfigParams implements Comparable<ConfigParams> {
         return this;
     }
 
+    public ConfigParams withSystemPropertiesOverride() {
+        this.withSystemPropertiesOverride = true;
+        return this;
+    }
+
     public ConfigParams includeAll() {
         this.includeAll = true;
         return this;
@@ -102,6 +112,15 @@ public class ConfigParams implements Comparable<ConfigParams> {
         return this;
     }
 
+    public ConfigParams withJdbc(Connection connection, String statement, String[] params) {
+        JdbcLookup jdbcLookup = new JdbcLookup();
+        jdbcLookup.connection = connection;
+        jdbcLookup.statement = statement;
+        jdbcLookup.params = params;
+        jdbcLookups.add(jdbcLookup);
+        return this;
+    }
+
     @Override
     public int compareTo(ConfigParams o) {
         return COMPARATOR.compare(this, o);
@@ -125,4 +144,10 @@ public class ConfigParams implements Comparable<ConfigParams> {
         Reader reader;
         String suffix;
     }
+
+    public static class JdbcLookup {
+        Connection connection;
+        String statement;
+        String[] params;
+    }
 }

content-config/src/test/java/org/xbib/content/config/test/ConfigLoaderTest.java

@@ -2,7 +2,7 @@ package org.xbib.content.config.test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import org.junit.jupiter.api.Test;
-import org.xbib.content.Settings;
+import org.xbib.settings.Settings;
 import org.xbib.content.config.ConfigLoader;
 import org.xbib.content.config.ConfigParams;
 import java.io.IOException;
@@ -29,4 +29,17 @@ public class ConfigLoaderTest {
         assertEquals("world2", settings.get("hello2"));
     }
 
+    @Test
+    public void testOverride() throws IOException {
+        System.setProperty("hello", "override");
+        Settings settings = ConfigLoader.getInstance()
+                .load(new ConfigParams()
+                        .withPath(null, null, "src/test/resources", "config.*"));
+        assertEquals("world", settings.get("hello"));
+        settings = ConfigLoader.getInstance()
+                .load(new ConfigParams()
+                        .withSystemPropertiesOverride()
+                        .withPath(null, null, "src/test/resources", "config.*"));
+        assertEquals("override", settings.get("hello"));
+    }
 }

content-config/src/test/resources/META-INF/services/org.xbib.content.SettingsLoader

@@ -1,3 +0,0 @@
-org.xbib.content.settings.datastructures.PropertiesSettingsLoader
-org.xbib.content.settings.datastructures.json.JsonSettingsLoader
-org.xbib.content.settings.datastructures.yaml.YamlSettingsLoader

content-config/src/test/resources/META-INF/services/org.xbib.settings.SettingsLoader

@@ -0,0 +1,3 @@
+org.xbib.settings.datastructures.PropertiesSettingsLoader
+org.xbib.settings.datastructures.json.JsonSettingsLoader
+org.xbib.settings.datastructures.yaml.YamlSettingsLoader

content-json/build.gradle

@@ -1,6 +1,5 @@
 dependencies {
     api project(':content-core')
-    api project(':content-settings')
     api "com.fasterxml.jackson.core:jackson-databind:${project.property('jackson.version')}"
     testImplementation("org.mockito:mockito-core:${project.property('mockito.version')}") {
         exclude group: 'org.hamcrest'

content-json/src/main/java/module-info.java

@@ -1,5 +1,4 @@
 import org.xbib.content.XContent;
-import org.xbib.content.SettingsLoader;
 
 module org.xbib.content.json {
     exports org.xbib.content.json;
@@ -9,8 +8,6 @@ module org.xbib.content.json {
     exports org.xbib.content.json.patch;
     exports org.xbib.content.json.pointer;
     requires org.xbib.content.core;
-    requires org.xbib.content.settings;
     requires com.fasterxml.jackson.databind;
     provides XContent with org.xbib.content.json.JsonXContent;
-    provides SettingsLoader with org.xbib.content.json.JsonSettingsLoader;
 }

content-json/src/main/resources/META-INF/services/org.xbib.content.SettingsLoader

@@ -1 +0,0 @@
-org.xbib.content.json.JsonSettingsLoader

content-settings-datastructures-json/src/main/java/module-info.java

@@ -1,10 +0,0 @@
-import org.xbib.content.SettingsLoader;
-import org.xbib.content.settings.datastructures.json.JsonSettingsLoader;
-
-module org.xbib.content.settings.datastructures.json {
-    exports org.xbib.content.settings.datastructures.json;
-    requires transitive org.xbib.content.settings.datastructures;
-    requires org.xbib.datastructures.json.tiny;
-    uses SettingsLoader;
-    provides SettingsLoader with JsonSettingsLoader;
-}

content-settings-datastructures-json/src/main/resources/META-INF.services/org.xbib.content.SettingsLoader

@@ -1 +0,0 @@
-org.xbib.content.settings.datastructures.json.JsonSettingsLoader

content-settings-datastructures-yaml/src/main/java/module-info.java

@@ -1,10 +0,0 @@
-import org.xbib.content.SettingsLoader;
-import org.xbib.content.settings.datastructures.yaml.YamlSettingsLoader;
-
-module org.xbib.content.settings.datastructures.yaml {
-    exports org.xbib.content.settings.datastructures.yaml;
-    requires transitive org.xbib.content.settings.datastructures;
-    requires org.xbib.datastructures.yaml.tiny;
-    uses SettingsLoader;
-    provides SettingsLoader with YamlSettingsLoader;
-}

content-settings-datastructures-yaml/src/main/resources/META-INF/services/org.xbib.content.settings.datastructures.SettingsLoader

@@ -1 +0,0 @@
-org.xbib.content.settings.datastructures.yaml.YamlSettingsLoader

content-settings-datastructures/src/main/java/module-info.java

@@ -1,15 +0,0 @@
-import org.xbib.content.SettingsBuilder;
-import org.xbib.content.SettingsLoader;
-import org.xbib.content.settings.datastructures.DatastructureSettingsBuilder;
-import org.xbib.content.settings.datastructures.PropertiesSettingsLoader;
-
-module org.xbib.content.settings.datastructures {
-    uses SettingsLoader;
-    provides SettingsLoader with PropertiesSettingsLoader;
-    uses SettingsBuilder;
-    provides SettingsBuilder with DatastructureSettingsBuilder;
-    exports org.xbib.content.settings.datastructures;
-    requires transitive  org.xbib.content.api;
-    requires org.xbib.datastructures.tiny;
-    requires transitive org.xbib.datastructures.api;
-}

content-settings-datastructures/src/main/resources/META-INF/services/org.xbib.content.SettingsBuilder

@@ -1 +0,0 @@
-org.xbib.content.settings.datastructures.DatastructureSettingsBuilder

content-settings-datastructures/src/main/resources/META-INF/services/org.xbib.content.SettingsLoader

@@ -1 +0,0 @@
-org.xbib.content.settings.datastructures.PropertiesSettingsLoader

content-settings/src/main/java/module-info.java

@@ -1,15 +0,0 @@
-import org.xbib.content.SettingsBuilder;
-import org.xbib.content.SettingsLoader;
-import org.xbib.content.settings.ContentSettingsBuilder;
-import org.xbib.content.settings.PropertiesSettingsLoader;
-
-module org.xbib.content.settings {
-    uses SettingsLoader;
-    provides SettingsLoader with PropertiesSettingsLoader;
-    uses SettingsBuilder;
-    provides SettingsBuilder with ContentSettingsBuilder;
-    exports org.xbib.content.settings;
-    requires org.xbib.content.core;
-    requires org.xbib.datastructures.api;
-    requires transitive org.xbib.datastructures.tiny;
-}

content-settings/src/main/java/org/xbib/content/settings/package-info.java

@@ -1,4 +0,0 @@
-/**
- * Classes for settings.
- */
-package org.xbib.content.settings;

content-settings/src/main/resources/META-INF/services/org.xbib.content.SettingsBuilder

@@ -1 +0,0 @@
-org.xbib.content.settings.ContentSettingsBuilder

content-settings/src/main/resources/META-INF/services/org.xbib.content.SettingsLoader

@@ -1 +0,0 @@
-org.xbib.content.settings.PropertiesSettingsLoader

content-yaml/build.gradle

@@ -1,6 +1,5 @@
 dependencies {
     api project(':content-core')
-    api project(':content-settings')
     api "com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:${project.property('jackson.version')}"
     implementation "org.yaml:snakeyaml:${project.property('snakeyaml.version')}"
 }

content-yaml/src/main/java/module-info.java

@@ -1,12 +1,9 @@
-import org.xbib.content.SettingsLoader;
 import org.xbib.content.XContent;
 
 module org.xbib.content.yaml {
   exports org.xbib.content.yaml;
   requires transitive org.xbib.content.core;
-  requires transitive org.xbib.content.settings;
   requires transitive com.fasterxml.jackson.dataformat.yaml;
   requires com.fasterxml.jackson.core;
   provides XContent with org.xbib.content.yaml.YamlXContent;
-  provides SettingsLoader with org.xbib.content.yaml.YamlSettingsLoader;
 }

content-yaml/src/main/resources/META-INF/services/org.xbib.content.SettingsLoader

@@ -1 +0,0 @@
-org.xbib.content.yaml.YamlSettingsLoader

settings-api/src/main/java/module-info.java

@@ -0,0 +1,10 @@
+import org.xbib.settings.SettingsBuilder;
+import org.xbib.settings.SettingsLoader;
+
+module org.xbib.settings.api {
+    exports org.xbib.settings;
+    uses SettingsBuilder;
+    uses SettingsLoader;
+    requires transitive org.xbib.datastructures.api;
+    requires transitive java.sql;
+}

settings-api/src/main/java/org/xbib/settings/PlaceholderResolver.java

@@ -1,4 +1,4 @@
-package org.xbib.content;
+package org.xbib.settings;
 
 /**
  * Strategy interface used to resolve replacement values for placeholders contained in Strings.

settings-api/src/main/java/org/xbib/settings/PropertyPlaceholder.java

@@ -1,4 +1,4 @@
-package org.xbib.content;
+package org.xbib.settings;
 
 import java.util.HashSet;
 import java.util.Set;

settings-api/src/main/java/org/xbib/settings/Settings.java

@@ -1,7 +1,8 @@
-package org.xbib.content;
+package org.xbib.settings;
 
 import org.xbib.datastructures.api.ByteSizeValue;
 import org.xbib.datastructures.api.TimeValue;
+import java.io.IOException;
 import java.util.Map;
 import java.util.Optional;
 import java.util.ServiceLoader;
@@ -62,4 +63,6 @@ public interface Settings extends AutoCloseable {
     Settings getByPrefix(String prefix);
 
     boolean containsSetting(String setting);
+
+    void close() throws IOException;
 }

settings-api/src/main/java/org/xbib/settings/SettingsBuilder.java

@@ -1,4 +1,4 @@
-package org.xbib.content;
+package org.xbib.settings;
 
 import java.io.InputStream;
 import java.nio.file.Path;
@@ -9,13 +9,12 @@ import java.sql.SQLException;
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.TimeUnit;
+import java.util.function.Function;
 
 public interface SettingsBuilder {
 
     SettingsBuilder put(String setting, String value);
 
-    SettingsBuilder put(String setting, Class<?> clazz);
-
     SettingsBuilder put(String setting, boolean value);
 
     SettingsBuilder put(String setting, int value);
@@ -41,9 +40,9 @@ public interface SettingsBuilder {
 
     SettingsBuilder loadFromResource(String resourceName, InputStream inputStream) throws SettingsException;
 
-    default SettingsBuilder fromJdbcConfTable(Connection connection, String id, String type) throws SQLException {
-        try (PreparedStatement statement = connection.prepareStatement("select key, value from conf where id = ? and type = ?",
-                new String[]{id, type}); ResultSet resultSet = statement.executeQuery()) {
+    default SettingsBuilder fromJdbc(Connection connection, String statement, String[] params) throws SQLException {
+        try (PreparedStatement preparedStatement = connection.prepareStatement(statement, params);
+             ResultSet resultSet = preparedStatement.executeQuery()) {
             while (resultSet.next()) {
                 String key = resultSet.getString("key");
                 String value = resultSet.getString("value");
@@ -57,17 +56,49 @@ public interface SettingsBuilder {
 
     SettingsBuilder loadFromSystemEnvironment();
 
+    /**
+     * Runs across all the settings set on this builder and replaces {@code ${...}} elements in the
+     * each setting value according to the following logic:
+     * First, tries to resolve it against a System property ({@link System#getProperty(String)}), next,
+     * tries and resolve it against an environment variable ({@link System#getenv(String)}), next,
+     * tries and resolve it against a date pattern to resolve the current date,
+     * and last, tries and replace it with another setting already set on this builder.
+     * @param propertyPlaceholder the property place holder
+     * @param placeholderResolver  the place holder resolver
+     * @return this builder
+     */
     SettingsBuilder replacePropertyPlaceholders(PropertyPlaceholder propertyPlaceholder,
                                                 PlaceholderResolver placeholderResolver);
 
+    /**
+     * A default method to replace property placeholders.
+     * @return this builder
+     */
     SettingsBuilder replacePropertyPlaceholders();
 
+    /**
+     * Optional settings refresh mechanism, using reloading from a path after a give time period.
+     * May not be implemented at all.
+     */
     SettingsBuilder setRefresh(Path path, long initialDelay, long period, TimeUnit timeUnit);
 
+    /**
+     * Map all settings keys and values to other keys and values.
+     * Example usage is to override settings from another priority source.
+     * @return this builder
+     */
+    SettingsBuilder map(Function<Map.Entry<String, String>, Map.Entry<String, String>> function);
+
+    /**
+     * Return the Settings from this SettingsBuilder.
+     * @return the settings
+     */
     Settings build();
 
+    /**
+     * Returns true if the settings builder is empty.
+     * @return true if empty
+     */
     boolean isEmpty();
 
-    Map<String, String> map();
-
 }

settings-api/src/main/java/org/xbib/settings/SettingsException.java

@@ -1,4 +1,4 @@
-package org.xbib.content;
+package org.xbib.settings;
 
 /**
  * A generic failure to handle settings.

settings-api/src/main/java/org/xbib/settings/SettingsLoader.java

@@ -1,4 +1,4 @@
-package org.xbib.content;
+package org.xbib.settings;
 
 import java.io.IOException;
 import java.util.Map;

settings-api/src/main/java/org/xbib/settings/SettingsLoaderService.java

@@ -1,6 +1,4 @@
-package org.xbib.content.settings.datastructures;
-
-import org.xbib.content.SettingsLoader;
+package org.xbib.settings;
 
 import java.util.HashMap;
 import java.util.HashSet;

settings-api/src/main/java/org/xbib/settings/package-info.java

@@ -0,0 +1,4 @@
+/**
+ * Settings API.
+ */
+package org.xbib.settings;

settings-content-json/build.gradle

@@ -0,0 +1,4 @@
+dependencies {
+    api project(':settings-content')
+    api project(':content-json')
+}

settings-content-json/src/main/java/module-info.java

@@ -0,0 +1,12 @@
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.content.json.JsonSettingsLoader;
+
+module org.xbib.settings.content.json {
+    exports org.xbib.settings.content.json;
+    requires transitive org.xbib.settings.content;
+    requires org.xbib.content.api;
+    requires org.xbib.content.json;
+    requires org.xbib.settings.api;
+    uses SettingsLoader;
+    provides SettingsLoader with JsonSettingsLoader;
+}

settings-content-json/src/main/java/org/xbib/settings/content/json/JsonSettingsLoader.java

@@ -1,7 +1,8 @@
-package org.xbib.content.json;
+package org.xbib.settings.content.json;
 
 import org.xbib.content.XContent;
-import org.xbib.content.settings.AbstractSettingsLoader;
+import org.xbib.content.json.JsonXContent;
+import org.xbib.settings.content.AbstractSettingsLoader;
 import java.util.Set;
 
 /**

settings-content-json/src/main/resources/META-INF/services/org.xbib.settings.SettingsLoader

@@ -0,0 +1 @@
+org.xbib.settings.content.json.JsonSettingsLoader

settings-content-yaml/build.gradle

@@ -0,0 +1,4 @@
+dependencies {
+    api project(':settings-content')
+    api project(':content-yaml')
+}

settings-content-yaml/src/main/java/module-info.java

@@ -0,0 +1,12 @@
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.content.yaml.YamlSettingsLoader;
+
+module org.xbib.settings.content.yaml {
+    exports org.xbib.settings.content.yaml;
+    requires transitive org.xbib.settings.content;
+    requires org.xbib.content.api;
+    requires org.xbib.content.yaml;
+    requires org.xbib.settings.api;
+    uses SettingsLoader;
+    provides SettingsLoader with YamlSettingsLoader;
+}

settings-content-yaml/src/main/java/org/xbib/settings/content/yaml/YamlSettingsLoader.java

@@ -1,7 +1,8 @@
-package org.xbib.content.yaml;
+package org.xbib.settings.content.yaml;
 
 import org.xbib.content.XContent;
-import org.xbib.content.settings.AbstractSettingsLoader;
+import org.xbib.content.yaml.YamlXContent;
+import org.xbib.settings.content.AbstractSettingsLoader;
 import java.io.IOException;
 import java.util.Map;
 import java.util.Set;

settings-content-yaml/src/main/resources/META-INF/resources/org.xbib.settings.SettingsLoader

@@ -0,0 +1 @@
+org.xbib.settings.content.yaml.YamlSettingsLoader

settings-content/build.gradle

@@ -1,5 +1,6 @@
 dependencies {
+    api project(':settings-api')
     api project(':content-core')
     api "org.xbib:datastructures-tiny:${project.property('xbib-datastructures.version')}"
-    testImplementation project(":content-json")
+    testImplementation project(":settings-content-json")
 }

settings-content/src/main/java/module-info.java

@@ -0,0 +1,16 @@
+import org.xbib.settings.SettingsBuilder;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.content.ContentSettingsBuilder;
+import org.xbib.settings.content.PropertiesSettingsLoader;
+
+module org.xbib.settings.content {
+    uses SettingsLoader;
+    provides SettingsLoader with PropertiesSettingsLoader;
+    uses SettingsBuilder;
+    provides SettingsBuilder with ContentSettingsBuilder;
+    exports org.xbib.settings.content;
+    requires org.xbib.settings.api;
+    requires org.xbib.content.core;
+    requires org.xbib.datastructures.api;
+    requires transitive org.xbib.datastructures.tiny;
+}

settings-content/src/main/java/org/xbib/settings/content/AbstractSettingsLoader.java

@@ -1,6 +1,6 @@
-package org.xbib.content.settings;
+package org.xbib.settings.content;
 
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.SettingsLoader;
 import org.xbib.content.XContent;
 import org.xbib.content.XContentBuilder;
 import org.xbib.content.core.DefaultXContentBuilder;

settings-content/src/main/java/org/xbib/settings/content/ContentSettings.java

@@ -1,7 +1,9 @@
-package org.xbib.content.settings;
+package org.xbib.settings.content;
 
-import org.xbib.content.Settings;
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.Settings;
+import org.xbib.settings.SettingsException;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.SettingsLoaderService;
 import org.xbib.datastructures.api.ByteSizeValue;
 import org.xbib.datastructures.api.TimeValue;
 import org.xbib.datastructures.tiny.TinyMap;
@@ -21,8 +23,6 @@ import java.util.concurrent.atomic.AtomicBoolean;
 
 public class ContentSettings implements Settings, AutoCloseable {
 
-    public static final ContentSettings EMPTY_SETTINGS = new ContentSettingsBuilder().build();
-
     public static final String[] EMPTY_ARRAY = new String[0];
 
     public static final int BUFFER_SIZE = 1024 * 4;
@@ -44,6 +44,10 @@ public class ContentSettings implements Settings, AutoCloseable {
         }
     }
 
+    public static ContentSettingsBuilder builder() {
+        return new ContentSettingsBuilder();
+    }
+
     public static ContentSettings readSettingsFromMap(Map<String, Object> map) {
         ContentSettingsBuilder builder = new ContentSettingsBuilder();
         for (Map.Entry<String, Object> entry : map.entrySet()) {
@@ -290,7 +294,7 @@ public class ContentSettings implements Settings, AutoCloseable {
     }
 
     @Override
-    public Map<String, org.xbib.content.Settings> getGroups(String prefix) {
+    public Map<String, Settings> getGroups(String prefix) {
         String settingPrefix = prefix;
         if (settingPrefix.charAt(settingPrefix.length() - 1) != '.') {
             settingPrefix = settingPrefix + ".";
@@ -313,7 +317,7 @@ public class ContentSettings implements Settings, AutoCloseable {
                 groupSettings.put(value, get(setting));
             }
         }
-        TinyMap.Builder<String, org.xbib.content.Settings> retVal = TinyMap.builder();
+        TinyMap.Builder<String, Settings> retVal = TinyMap.builder();
         for (Map.Entry<String, Map<String, String>> entry : hashMap.entrySet()) {
             retVal.put(entry.getKey(), new ContentSettings(entry.getValue()));
         }
@@ -416,14 +420,11 @@ public class ContentSettings implements Settings, AutoCloseable {
 
         private final AtomicBoolean closed;
 
-        private final SettingsLoaderService settingsLoaderService;
-
         DefaultSettingsRefresher(Path path, long initialDelay, long period, TimeUnit timeUnit) {
             this.path = path;
             this.executorService = Executors.newSingleThreadScheduledExecutor();
             executorService.scheduleAtFixedRate(this, initialDelay, period, timeUnit);
             this.closed = new AtomicBoolean();
-            this.settingsLoaderService = new SettingsLoaderService();
         }
 
         @Override
@@ -431,7 +432,7 @@ public class ContentSettings implements Settings, AutoCloseable {
             try {
                 if (!closed.get()) {
                     String settingsSource = Files.readString(path);
-                    SettingsLoader settingsLoader = settingsLoaderService.loaderFromResource(path.toString());
+                    SettingsLoader settingsLoader = SettingsLoaderService.getInstance().loaderFromResource(path.toString());
                     map = settingsLoader.load(settingsSource);
                 }
             } catch (IOException e) {

settings-content/src/main/java/org/xbib/settings/content/ContentSettingsBuilder.java

@@ -1,10 +1,12 @@
-package org.xbib.content.settings;
-
-import org.xbib.content.PlaceholderResolver;
-import org.xbib.content.PropertyPlaceholder;
-import org.xbib.content.Settings;
-import org.xbib.content.SettingsBuilder;
-import org.xbib.content.SettingsLoader;
+package org.xbib.settings.content;
+
+import org.xbib.settings.PlaceholderResolver;
+import org.xbib.settings.PropertyPlaceholder;
+import org.xbib.settings.Settings;
+import org.xbib.settings.SettingsBuilder;
+import org.xbib.settings.SettingsException;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.SettingsLoaderService;
 import org.xbib.datastructures.tiny.TinyMap;
 
 import java.io.IOException;
@@ -19,12 +21,15 @@ import java.time.format.DateTimeFormatter;
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.TimeUnit;
+import java.util.function.Function;
 
 /**
  *
  */
 public class ContentSettingsBuilder implements SettingsBuilder {
 
+    private final SettingsLoaderService settingsLoaderService = SettingsLoaderService.getInstance();
+
     private final Map<String, String> map;
 
     private Path path;
@@ -39,18 +44,10 @@ public class ContentSettingsBuilder implements SettingsBuilder {
         map = TinyMap.builder();
     }
 
-    public Map<String, String> map() {
-        return map;
-    }
-
     public String remove(String key) {
         return map.remove(key);
     }
 
-    public String get(String key) {
-        return map.get(key);
-    }
-
     @Override
     public boolean isEmpty() {
         return map.isEmpty();
@@ -238,7 +235,6 @@ public class ContentSettingsBuilder implements SettingsBuilder {
      * @return builder
      */
     public ContentSettingsBuilder loadFromString(String source) {
-        SettingsLoaderService settingsLoaderService = new SettingsLoaderService();
         SettingsLoader settingsLoader = settingsLoaderService.loaderFromString(source);
         try {
             Map<String, String> loadedSettings = settingsLoader.load(source);
@@ -272,7 +268,6 @@ public class ContentSettingsBuilder implements SettingsBuilder {
      */
     @Override
     public ContentSettingsBuilder loadFromResource(String resourceName, InputStream inputStream) throws SettingsException {
-        SettingsLoaderService settingsLoaderService = new SettingsLoaderService();
         SettingsLoader settingsLoader = settingsLoaderService.loaderFromResource(resourceName);
         try {
             Map<String, String> loadedSettings = settingsLoader
@@ -357,6 +352,12 @@ public class ContentSettingsBuilder implements SettingsBuilder {
         return this;
     }
 
+    @Override
+    public SettingsBuilder map(Function<Map.Entry<String, String>, Map.Entry<String, String>> function) {
+        map.entrySet().stream().map(function).forEach(e -> put(e.getKey(), e.getValue()));
+        return this;
+    }
+
     public ContentSettings build() {
         return new ContentSettings(map, path, initialDelay, period, timeUnit);
     }

settings-content/src/main/java/org/xbib/settings/content/PropertiesSettingsLoader.java

@@ -1,7 +1,7 @@
-package org.xbib.content.settings;
+package org.xbib.settings.content;
 
 import org.xbib.content.io.BytesReference;
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.SettingsLoader;
 import java.io.IOException;
 import java.io.InputStreamReader;
 import java.io.Reader;

settings-content/src/main/java/org/xbib/settings/content/package-info.java

@@ -0,0 +1,4 @@
+/**
+ * Classes for settings.
+ */
+package org.xbib.settings.content;

settings-content/src/main/resources/META-INF/services/org.xbib.settings.SettingsBuilder

@@ -0,0 +1 @@
+org.xbib.settings.content.ContentSettingsBuilder

settings-content/src/main/resources/META-INF/services/org.xbib.settings.SettingsLoader

@@ -0,0 +1 @@
+org.xbib.settings.content.PropertiesSettingsLoader

settings-content/src/test/java/org/xbib/settings/content/test/SettingsTest.java

@@ -1,16 +1,16 @@
-package org.xbib.content.settings.test;
+package org.xbib.settings.content.test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import org.junit.jupiter.api.Test;
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.SettingsLoader;
 import org.xbib.content.core.XContentHelper;
 import org.xbib.content.io.BytesArray;
 import org.xbib.content.io.BytesReference;
-import org.xbib.content.json.JsonSettingsLoader;
+import org.xbib.settings.content.json.JsonSettingsLoader;
 import org.xbib.content.json.JsonXContent;
-import org.xbib.content.Settings;
+import org.xbib.settings.Settings;
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.StringReader;

settings-datastructures-json/build.gradle

@@ -1,4 +1,4 @@
 dependencies {
-    api project(':content-settings-datastructures')
+    api project(':settings-datastructures')
     api "org.xbib:datastructures-json-tiny:${project.property('xbib-datastructures.version')}"
 }

settings-datastructures-json/src/main/java/module-info.java

@@ -0,0 +1,10 @@
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.datastructures.json.JsonSettingsLoader;
+
+module org.xbib.settings.datastructures.json {
+    exports org.xbib.settings.datastructures.json;
+    requires transitive org.xbib.settings.datastructures;
+    requires org.xbib.datastructures.json.tiny;
+    uses SettingsLoader;
+    provides SettingsLoader with JsonSettingsLoader;
+}

settings-datastructures-json/src/main/java/org/xbib/settings/datastructures/json/JsonSettingsLoader.java

@@ -1,6 +1,6 @@
-package org.xbib.content.settings.datastructures.json;
+package org.xbib.settings.datastructures.json;
 
-import org.xbib.content.settings.datastructures.AbstractSettingsLoader;
+import org.xbib.settings.datastructures.AbstractSettingsLoader;
 import org.xbib.datastructures.api.DataStructure;
 import org.xbib.datastructures.json.tiny.Json;
 import java.util.Set;

settings-datastructures-json/src/main/java/org/xbib/settings/datastructures/json/package-info.java

@@ -1,4 +1,4 @@
 /**
  * JSON settings with the datastructures package.
  */
-package org.xbib.content.settings.datastructures.json;
+package org.xbib.settings.datastructures.json;

settings-datastructures-json/src/main/resources/META-INF.services/org.xbib.settings.SettingsLoader

@@ -0,0 +1 @@
+org.xbib.settings.datastructures.json.JsonSettingsLoader

settings-datastructures-json/src/test/java/org/xbib/settings/datastructures/json/test/JsonSettingsTest.java

@@ -1,9 +1,9 @@
-package org.xbib.content.settings.datastructures.json.test;
+package org.xbib.settings.datastructures.json.test;
 
 import org.junit.jupiter.api.Test;
-import org.xbib.content.Settings;
-import org.xbib.content.SettingsLoader;
-import org.xbib.content.settings.datastructures.json.JsonSettingsLoader;
+import org.xbib.settings.Settings;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.datastructures.json.JsonSettingsLoader;
 import java.io.IOException;
 import java.util.Arrays;
 import java.util.HashMap;

settings-datastructures-json/src/test/java/org/xbib/settings/datastructures/json/test/package-info.java

@@ -1,4 +1,4 @@
 /**
  * Testing JSON settings with the datastructures package.
  */
-package org.xbib.content.settings.datastructures.json.test;
+package org.xbib.settings.datastructures.json.test;

settings-datastructures-yaml/build.gradle

@@ -1,4 +1,4 @@
 dependencies {
-    api project(':content-settings-datastructures')
+    api project(':settings-datastructures')
     api "org.xbib:datastructures-yaml-tiny:${project.property('xbib-datastructures.version')}"
 }

settings-datastructures-yaml/src/main/java/module-info.java

@@ -0,0 +1,10 @@
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.datastructures.yaml.YamlSettingsLoader;
+
+module org.xbib.settings.datastructures.yaml {
+    exports org.xbib.settings.datastructures.yaml;
+    requires transitive org.xbib.settings.datastructures;
+    requires org.xbib.datastructures.yaml.tiny;
+    uses SettingsLoader;
+    provides SettingsLoader with YamlSettingsLoader;
+}

settings-datastructures-yaml/src/main/java/org/xbib/settings/datastructures/yaml/YamlSettingsLoader.java

@@ -1,6 +1,6 @@
-package org.xbib.content.settings.datastructures.yaml;
+package org.xbib.settings.datastructures.yaml;
 
-import org.xbib.content.settings.datastructures.AbstractSettingsLoader;
+import org.xbib.settings.datastructures.AbstractSettingsLoader;
 import org.xbib.datastructures.api.DataStructure;
 import org.xbib.datastructures.yaml.tiny.Yaml;
 

settings-datastructures-yaml/src/main/java/org/xbib/settings/datastructures/yaml/package-info.java

@@ -1,4 +1,4 @@
 /**
  * YAML settings with the datastructures package.
  */
-package org.xbib.content.settings.datastructures.yaml;
+package org.xbib.settings.datastructures.yaml;

settings-datastructures-yaml/src/main/resources/META-INF/services/org.xbib.content.settings.datastructures.SettingsLoader

@@ -0,0 +1 @@
+org.xbib.settings.datastructures.yaml.YamlSettingsLoader

settings-datastructures-yaml/src/test/java/org/xbib/settings/datastructures/yaml/test/YamlSettingsTest.java

@@ -1,11 +1,11 @@
-package org.xbib.content.settings.datastructures.yaml.test;
+package org.xbib.settings.datastructures.yaml.test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import org.junit.jupiter.api.Test;
-import org.xbib.content.Settings;
-import org.xbib.content.SettingsLoader;
-import org.xbib.content.settings.datastructures.yaml.YamlSettingsLoader;
+import org.xbib.settings.Settings;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.datastructures.yaml.YamlSettingsLoader;
 import java.io.IOException;
 import java.util.Arrays;
 import java.util.HashMap;

settings-datastructures-yaml/src/test/java/org/xbib/settings/datastructures/yaml/test/package-info.java

@@ -1,4 +1,4 @@
 /**
  * Testing YAML settings with the datastructures package.
  */
-package org.xbib.content.settings.datastructures.yaml.test;
+package org.xbib.settings.datastructures.yaml.test;

settings-datastructures/build.gradle

@@ -1,4 +1,4 @@
 dependencies {
-    api project(':content-api')
+    api project(':settings-api')
     api "org.xbib:datastructures-tiny:${project.property('xbib-datastructures.version')}"
 }

settings-datastructures/src/main/java/module-info.java

@@ -0,0 +1,15 @@
+import org.xbib.settings.SettingsBuilder;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.datastructures.DatastructureSettingsBuilder;
+import org.xbib.settings.datastructures.PropertiesSettingsLoader;
+
+module org.xbib.settings.datastructures {
+    uses SettingsLoader;
+    provides SettingsLoader with PropertiesSettingsLoader;
+    uses SettingsBuilder;
+    provides SettingsBuilder with DatastructureSettingsBuilder;
+    exports org.xbib.settings.datastructures;
+    requires transitive org.xbib.settings.api;
+    requires org.xbib.datastructures.tiny;
+    requires transitive org.xbib.datastructures.api;
+}

settings-datastructures/src/main/java/org/xbib/settings/datastructures/AbstractSettingsLoader.java

@@ -1,6 +1,6 @@
-package org.xbib.content.settings.datastructures;
+package org.xbib.settings.datastructures;
 
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.SettingsLoader;
 import org.xbib.datastructures.api.Builder;
 import org.xbib.datastructures.api.DataStructure;
 import org.xbib.datastructures.api.ListNode;

settings-datastructures/src/main/java/org/xbib/settings/datastructures/DatastructureSettings.java

@@ -1,7 +1,7 @@
-package org.xbib.content.settings.datastructures;
+package org.xbib.settings.datastructures;
 
-import org.xbib.content.Settings;
-import org.xbib.content.SettingsException;
+import org.xbib.settings.Settings;
+import org.xbib.settings.SettingsException;
 import org.xbib.datastructures.api.ByteSizeValue;
 import org.xbib.datastructures.api.TimeValue;
 import org.xbib.datastructures.tiny.TinyMap;
@@ -21,6 +21,10 @@ public class DatastructureSettings implements Settings {
         this.map = map;
     }
 
+    public static DatastructureSettingsBuilder builder() {
+        return new DatastructureSettingsBuilder();
+    }
+
     public static DatastructureSettings fromMap(Map<String, Object> map) {
         DatastructureSettingsBuilder builder = new DatastructureSettingsBuilder();
         for (Map.Entry<String, Object> entry : map.entrySet()) {
@@ -233,7 +237,7 @@ public class DatastructureSettings implements Settings {
     }
 
     @Override
-    public Map<String, org.xbib.content.Settings> getGroups(String prefix) {
+    public Map<String, Settings> getGroups(String prefix) {
         String settingPrefix = prefix;
         if (settingPrefix.charAt(settingPrefix.length() - 1) != '.') {
             settingPrefix = settingPrefix + ".";
@@ -255,7 +259,7 @@ public class DatastructureSettings implements Settings {
                 groupSettings.put(value, get(o));
             }
         }
-        TinyMap.Builder<String, org.xbib.content.Settings> retVal = TinyMap.builder();
+        TinyMap.Builder<String, Settings> retVal = TinyMap.builder();
         for (String key : hashMap.keySet()) {
             TinyMap.Builder<String, String> value = hashMap.get(key);
             retVal.put(key, new DatastructureSettings(value.build()));

settings-datastructures/src/main/java/org/xbib/settings/datastructures/DatastructureSettingsBuilder.java

@@ -1,11 +1,12 @@
-package org.xbib.content.settings.datastructures;
+package org.xbib.settings.datastructures;
 
-import org.xbib.content.PlaceholderResolver;
-import org.xbib.content.PropertyPlaceholder;
-import org.xbib.content.Settings;
-import org.xbib.content.SettingsBuilder;
-import org.xbib.content.SettingsException;
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.PlaceholderResolver;
+import org.xbib.settings.PropertyPlaceholder;
+import org.xbib.settings.Settings;
+import org.xbib.settings.SettingsBuilder;
+import org.xbib.settings.SettingsException;
+import org.xbib.settings.SettingsLoader;
+import org.xbib.settings.SettingsLoaderService;
 import org.xbib.datastructures.tiny.TinyMap;
 
 import java.io.BufferedReader;
@@ -19,6 +20,7 @@ import java.time.format.DateTimeFormatter;
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.TimeUnit;
+import java.util.function.Function;
 import java.util.stream.Collectors;
 
 /**
@@ -34,18 +36,10 @@ public class DatastructureSettingsBuilder implements SettingsBuilder {
         map = TinyMap.builder();
     }
 
-    public Map<String, String> map() {
-        return map;
-    }
-
     public String remove(String key) {
         return map.remove(key);
     }
 
-    public String get(String key) {
-        return map.get(key);
-    }
-
     @Override
     public boolean isEmpty() {
         return map.isEmpty();
@@ -64,19 +58,6 @@ public class DatastructureSettingsBuilder implements SettingsBuilder {
         return this;
     }
 
-    /**
-     * Sets a setting with the provided setting key and class as value.
-     *
-     * @param key   The setting key
-     * @param clazz The setting class value
-     * @return The builder
-     */
-    @Override
-    public DatastructureSettingsBuilder put(String key, Class<?> clazz) {
-        map.put(key, clazz.getName());
-        return this;
-    }
-
     /**
      * Sets the setting with the provided setting key and the boolean value.
      *
@@ -308,17 +289,6 @@ public class DatastructureSettingsBuilder implements SettingsBuilder {
         return this;
     }
 
-    /**
-     * Runs across all the settings set on this builder and replaces {@code ${...}} elements in the
-     * each setting value according to the following logic:
-     * <p>
-     * First, tries to resolve it against a System property ({@link System#getProperty(String)}), next,
-     * tries and resolve it against an environment variable ({@link System#getenv(String)}), next,
-     * tries and resolve it against a date pattern to resolve the current date,
-     * and last, tries and replace it with another setting already set on this builder.
-     *
-     * @return builder
-     */
     @Override
     public DatastructureSettingsBuilder replacePropertyPlaceholders() {
         return replacePropertyPlaceholders(new PropertyPlaceholder("${", "}", false),
@@ -348,6 +318,12 @@ public class DatastructureSettingsBuilder implements SettingsBuilder {
         return this;
     }
 
+    @Override
+    public SettingsBuilder map(Function<Map.Entry<String, String>, Map.Entry<String, String>> function) {
+        map.entrySet().stream().map(function).forEach(e -> put(e.getKey(), e.getValue()));
+        return this;
+    }
+
     @Override
     public DatastructureSettings build() {
         return new DatastructureSettings(map.build());

settings-datastructures/src/main/java/org/xbib/settings/datastructures/PropertiesSettingsLoader.java

@@ -1,6 +1,6 @@
-package org.xbib.content.settings.datastructures;
+package org.xbib.settings.datastructures;
 
-import org.xbib.content.SettingsLoader;
+import org.xbib.settings.SettingsLoader;
 import org.xbib.datastructures.tiny.TinyMap;
 import java.io.IOException;
 import java.io.StringReader;

settings-datastructures/src/main/java/org/xbib/settings/datastructures/package-info.java

@@ -1,4 +1,4 @@
 /**
  * Classes for settings using the datastructures API.
  */
-package org.xbib.content.settings.datastructures;
+package org.xbib.settings.datastructures;

settings-datastructures/src/main/resources/META-INF/services/org.xbib.settings.SettingsBuilder

@@ -0,0 +1 @@
+org.xbib.settings.datastructures.DatastructureSettingsBuilder

settings-datastructures/src/main/resources/META-INF/services/org.xbib.settings.SettingsLoader

@@ -0,0 +1 @@
+org.xbib.settings.datastructures.PropertiesSettingsLoader

settings-datastructures/src/test/java/org/xbib/settings/datastructures/test/SettingsTest.java

@@ -1,7 +1,7 @@
-package org.xbib.content.settings.datastructures.test;
+package org.xbib.settings.datastructures.test;
 
 import org.junit.jupiter.api.Test;
-import org.xbib.content.Settings;
+import org.xbib.settings.Settings;
 
 import java.io.ByteArrayInputStream;
 import java.nio.charset.StandardCharsets;

settings.gradle

@@ -6,10 +6,13 @@ include 'content-language'
 include 'content-json'
 include 'content-rdf'
 include 'content-resource'
-include 'content-settings'
 include 'content-smile'
 include 'content-xml'
 include 'content-yaml'
-include 'content-settings-datastructures'
-include 'content-settings-datastructures-json'
-include 'content-settings-datastructures-yaml'
+include 'settings-api'
+include 'settings-content'
+include 'settings-content-json'
+include 'settings-content-yaml'
+include 'settings-datastructures'
+include 'settings-datastructures-json'
+include 'settings-datastructures-yaml'