From 1b6f48ef66c0d75f53c486a1e035df8fa1dd379c Mon Sep 17 00:00:00 2001 From: "chao.chang" Date: Fri, 22 Mar 2019 18:12:36 +0900 Subject: [PATCH] Adds log.isDebugEnabled() before log.debug(). --- .../redis/TraceRedisAutoConfiguration.java | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/redis/TraceRedisAutoConfiguration.java b/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/redis/TraceRedisAutoConfiguration.java index f8ed5bfe1..996a3e595 100644 --- a/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/redis/TraceRedisAutoConfiguration.java +++ b/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/redis/TraceRedisAutoConfiguration.java @@ -79,12 +79,16 @@ class TraceLettuceClientResourcesBeanPostProcessor implements BeanPostProcessor if (bean instanceof ClientResources) { ClientResources cr = (ClientResources) bean; if (!cr.tracing().isEnabled()) { - log.debug( - "Lettuce ClientResources bean is auto-configured to enable tracing."); + if (log.isDebugEnabled()) { + log.debug( + "Lettuce ClientResources bean is auto-configured to enable tracing."); + } return cr.mutate().tracing(BraveTracing.create(this.tracing)).build(); } - log.debug( - "Lettuce ClientResources bean is skipped for auto-configuration because tracing was already enabled."); + if (log.isDebugEnabled()) { + log.debug( + "Lettuce ClientResources bean is skipped for auto-configuration because tracing was already enabled."); + } } return bean; }