Remove some redundant isDebugEnabled() checks

SLF4J already performs these checks.
wzq1022 · Nov 6, 2017 · b6c97d9 · b6c97d9
1 parent f7d01cb
commit b6c97d9
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/eureka-client/src/main/java/com/netflix/discovery/shared/resolver/StaticClusterResolver.java b/eureka-client/src/main/java/com/netflix/discovery/shared/resolver/StaticClusterResolver.java
@@ -43,9 +43,7 @@ public StaticClusterResolver(String region, T... eurekaEndpoints) {
     public StaticClusterResolver(String region, List<T> eurekaEndpoints) {
         this.eurekaEndpoints = eurekaEndpoints;
         this.region = region;
-        if (logger.isDebugEnabled()) {
-            logger.debug("Fixed resolver configuration: {}", eurekaEndpoints);
-        }
+        logger.debug("Fixed resolver configuration: {}", eurekaEndpoints);
     }
 
     @Override

diff --git a/...client/src/main/java/com/netflix/discovery/shared/resolver/aws/ConfigClusterResolver.java b/...client/src/main/java/com/netflix/discovery/shared/resolver/aws/ConfigClusterResolver.java
@@ -86,9 +86,7 @@ private List<AwsEndpoint> getClusterEndpointsFromConfig() {
             }
         }
 
-        if (logger.isDebugEnabled()) {
-            logger.debug("Config resolved to {}", endpoints);
-        }
+        logger.debug("Config resolved to {}", endpoints);
 
         if (endpoints.isEmpty()) {
             logger.error("Cannot resolve to any endpoints from provided configuration: {}", serviceUrls);

diff --git a/.../src/main/java/com/netflix/discovery/shared/resolver/aws/ZoneAffinityClusterResolver.java b/.../src/main/java/com/netflix/discovery/shared/resolver/aws/ZoneAffinityClusterResolver.java
@@ -63,9 +63,7 @@ public List<AwsEndpoint> getClusterEndpoints() {
             Collections.reverse(randomizedList);
         }
 
-        if (logger.isDebugEnabled()) {
-            logger.debug("Local zone={}; resolved to: {}", myZone, randomizedList);
-        }
+        logger.debug("Local zone={}; resolved to: {}", myZone, randomizedList);
 
         return randomizedList;
     }