Merge pull request #424 from elandau/bugfix/default_config

config: Ensure dynamic config is always loaded for defaults

ribbon-core/src/main/java/com/netflix/client/config/ClientConfigFactory.java

@@ -34,7 +34,8 @@ public interface ClientConfigFactory {
         return StreamSupport.stream(ServiceLoader.load(ClientConfigFactory.class).spliterator(), false)
                 .sorted(Comparator
                         .comparingInt(ClientConfigFactory::getPriority)
-                        .thenComparing(Comparator.comparing(f -> f.getClass().getCanonicalName())))
+                        .thenComparing(f -> f.getClass().getCanonicalName())
+                        .reversed())
                 .findFirst()
                 .orElseGet(() -> {
                     throw new IllegalStateException("Expecting at least one implementation of ClientConfigFactory discoverable via the ServiceLoader");

ribbon-core/src/main/java/com/netflix/client/config/ReloadableClientConfig.java

@@ -393,15 +393,13 @@ public abstract class ReloadableClientConfig implements IClientConfig {
     public final <T> IClientConfig set(IClientConfigKey<T> key, T value) {
         Preconditions.checkArgument(key != null, "key cannot be null");
 
-        // Make sure the value is property typed
+        // Make sure the value is property typed.
         value = resolveValueToType(key, value);
 
-        // If a client is already specified then check if there's a dynamic config override available
-        if (isLoaded) {
-            value = resolveFinalProperty(key).orElse(value);
-        }
+        // Check if there's a dynamic config override available
+        value = resolveFinalProperty(key).orElse(value);
 
-        // Cache this new state
+        // Cache the new value
         internalProperties.put(key, Optional.ofNullable(value));
 
         // If this is the first time a property is seen and a client name has been specified then make sure