commit 995f3d8d07d71bcb6abfca2eac0e30710cfba757
author Mark Struberg <struberg@apache.org> Wed Mar 20 19:15:21 2019 +0100
committer Mark Struberg <struberg@apache.org> Wed Mar 20 19:15:21 2019 +0100
tree be4b3fbdf72e413190bc4aa3ff1356496601dd86
parent 6b63605912f4daf28f719109c3fc551c53e52787

rename to match ConfigJSR

impl/src/main/java/org/apache/geronimo/config/ConfigAccessorImpl.java

diff --git a/impl/src/main/java/org/apache/geronimo/config/ConfigValueImpl.java b/impl/src/main/java/org/apache/geronimo/config/ConfigAccessorImpl.java
similarity index 94%
rename from impl/src/main/java/org/apache/geronimo/config/ConfigValueImpl.java
rename to impl/src/main/java/org/apache/geronimo/config/ConfigAccessorImpl.java
index 5c663f8..b374151 100644
--- a/impl/src/main/java/org/apache/geronimo/config/ConfigValueImpl.java
+++ b/impl/src/main/java/org/apache/geronimo/config/ConfigAccessorImpl.java
@@ -33,8 +33,8 @@
  * @author <a href="mailto:struberg@apache.org">Mark Struberg</a>
  */
 @Typed
-public class ConfigValueImpl<T> implements ConfigAccessor<T>, ConfigAccessor.Builder<T> {
-    private static final Logger logger = Logger.getLogger(ConfigValueImpl.class.getName());
+public class ConfigAccessorImpl<T> implements ConfigAccessor<T>, ConfigAccessor.Builder<T> {
+    private static final Logger logger = Logger.getLogger(ConfigAccessorImpl.class.getName());
 
     private final ConfigImpl config;
 
@@ -62,7 +62,7 @@
      */
     private Converter<T> converter;
 
-    public ConfigValueImpl(ConfigImpl config, String key, Class<T> type) {
+    public ConfigAccessorImpl(ConfigImpl config, String key, Class<T> type) {
         this.config = config;
         this.keyOriginal = key;
         this.configEntryType = type;
@@ -106,18 +106,19 @@
     }
 
     @Override
-    public ConfigValueImpl<T> cacheFor(Duration timeUnit) {
+    public ConfigAccessorImpl<T> cacheFor(Duration timeUnit) {
         this.cacheTimeNs = timeUnit.toNanos();
         return this;
     }
 
     @Override
-    public ConfigValueImpl<T> evaluateVariables(boolean evaluateVariables) {
+    public ConfigAccessorImpl<T> evaluateVariables(boolean evaluateVariables) {
         this.evaluateVariables = evaluateVariables;
         return this;
     }
 
-    public ConfigValueImpl<T> addLookupSuffix(String suffixValue) {
+    @Override
+    public ConfigAccessorImpl<T> addLookupSuffix(String suffixValue) {
         if (lookupChain == null) {
             lookupChain = new ArrayList<>();
         }
@@ -125,7 +126,8 @@
         return this;
     }
 
-    public ConfigValueImpl<T> addLookupSuffix(ConfigAccessor<String> suffixAccessor) {
+    @Override
+    public ConfigAccessorImpl<T> addLookupSuffix(ConfigAccessor<String> suffixAccessor) {
         if (lookupChain == null) {
             lookupChain = new ArrayList<>();
         }
@@ -140,7 +142,7 @@
 
     //X will later get added again @Override
     /*X
-    public ConfigValueImpl<T> onChange(ConfigChanged valueChangeListener) {
+    public ConfigAccessorImpl<T> onChange(ConfigChanged valueChangeListener) {
         this.valueChangeListener = valueChangeListener;
         return this;
     }

impl/src/main/java/org/apache/geronimo/config/ConfigImpl.java

diff --git a/impl/src/main/java/org/apache/geronimo/config/ConfigImpl.java b/impl/src/main/java/org/apache/geronimo/config/ConfigImpl.java
index 0a9b31d..7ad8097 100644
--- a/impl/src/main/java/org/apache/geronimo/config/ConfigImpl.java
+++ b/impl/src/main/java/org/apache/geronimo/config/ConfigImpl.java
@@ -202,8 +202,8 @@
         return converter;
     }
 
-    public <T> ConfigValueImpl<T> access(String key, Class<T> type) {
-        return new ConfigValueImpl<T>(this, key, type);
+    public <T> ConfigAccessorImpl<T> access(String key, Class<T> type) {
+        return new ConfigAccessorImpl<T>(this, key, type);
     }
 
     @Override