diff --git a/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/deployer/BackingServicesProvisionService.java b/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/deployer/BackingServicesProvisionService.java index 98679ed..5d13c11 100644 --- a/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/deployer/BackingServicesProvisionService.java +++ b/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/deployer/BackingServicesProvisionService.java @@ -51,10 +51,10 @@ public class BackingServicesProvisionService { .parallel() .runOn(Schedulers.parallel()) .flatMap(deployerClient::updateServiceInstance) - .doOnRequest(l -> log.info("Updating backing services {}", backingServices)) - .doOnEach(response -> log.info("Finished updating backing service {}", response)) - .doOnComplete(() -> log.info("Finished updating backing service {}", backingServices)) - .doOnError(exception -> log.info("Error updating backing services {} with error '{}'", + .doOnRequest(l -> log.debug("Updating backing services {}", backingServices)) + .doOnEach(response -> log.debug("Finished updating backing service {}", response)) + .doOnComplete(() -> log.debug("Finished updating backing service {}", backingServices)) + .doOnError(exception -> log.error("Error updating backing services {} with error '{}'", backingServices, exception.getMessage())) .sequential(); } diff --git a/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/extensions/support/KebabCasePropertyBeanIntrospector.java b/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/extensions/support/KebabCasePropertyBeanIntrospector.java index 35e5664..3ac19b7 100644 --- a/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/extensions/support/KebabCasePropertyBeanIntrospector.java +++ b/spring-cloud-app-broker-core/src/main/java/org/springframework/cloud/appbroker/extensions/support/KebabCasePropertyBeanIntrospector.java @@ -18,8 +18,8 @@ package org.springframework.cloud.appbroker.extensions.support; import org.apache.commons.beanutils.BeanIntrospector; import org.apache.commons.beanutils.DefaultBeanIntrospector; import org.apache.commons.beanutils.IntrospectionContext; -import org.apache.commons.logging.Log; -import org.apache.commons.logging.LogFactory; +import reactor.util.Logger; +import reactor.util.Loggers; import java.beans.IntrospectionException; import java.beans.Introspector; @@ -41,7 +41,7 @@ public class KebabCasePropertyBeanIntrospector implements BeanIntrospector { private static final String WRITE_METHOD_PREFIX = "set"; - private final Log log = LogFactory.getLog(getClass()); + private final Logger log = Loggers.getLogger(getClass()); /** * Creates a new instance of KebabCaseBeanIntrospector and @@ -68,9 +68,8 @@ public class KebabCasePropertyBeanIntrospector implements BeanIntrospector { context.addPropertyDescriptor(createPropertyDescriptor(m)); } } catch (final IntrospectionException e) { - log.info("Error when creating PropertyDescriptor for " + m - + "! Ignoring this property."); - log.debug("Exception is:", e); + log.error("Error when creating PropertyDescriptor for method '{}'. " + + "This property will be ignored. {}", m, e); } } }