Change log level info->debug in obvious cases in SpringPlugin

HotswapProjects · Sep 2, 2024 · 85bbaa5 · 85bbaa5
1 parent 419359f
commit 85bbaa5
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/...pring-plugin/src/main/java/org/hotswap/agent/plugin/spring/core/BeanFactoryProcessor.java b/...pring-plugin/src/main/java/org/hotswap/agent/plugin/spring/core/BeanFactoryProcessor.java
@@ -61,7 +61,7 @@ public static void postProcessDestroySingleton(DefaultSingletonBeanRegistry bean
         // check if reload , then log
         if (beanFactory instanceof ConfigurableListableBeanFactory &&
             BeanFactoryAssistant.getBeanFactoryAssistant((ConfigurableListableBeanFactory)beanFactory).isReload()) {
-            LOGGER.info("destroy bean '{}' from '{}'", beanName, ObjectUtils.identityToString(beanFactory));
+            LOGGER.debug("destroy bean '{}' from '{}'", beanName, ObjectUtils.identityToString(beanFactory));
         }
         if (beanFactory instanceof BeanFactoryLifecycle) {
             ((BeanFactoryLifecycle)beanFactory).hotswapAgent$destroyBean(beanName);
@@ -80,7 +80,7 @@ public static void postProcessCreateBean(AbstractAutowireCapableBeanFactory bean
         if (beanFactory instanceof ConfigurableListableBeanFactory
             && BeanFactoryAssistant.getBeanFactoryAssistant((ConfigurableListableBeanFactory)beanFactory).isReload()
             && mbd.isSingleton()) {
-            LOGGER.info("create new singleton bean '{}' from '{}'", beanName,
+            LOGGER.debug("create new singleton bean '{}' from '{}'", beanName,
                 ObjectUtils.identityToString(beanFactory));
         }
     }

diff --git a/...ng-plugin/src/main/java/org/hotswap/agent/plugin/spring/getbean/DetachableBeanHolder.java b/...ng-plugin/src/main/java/org/hotswap/agent/plugin/spring/getbean/DetachableBeanHolder.java
@@ -83,7 +83,7 @@ public static void detachBeans() {
             }
         }
         if (i > 0) {
-            LOGGER.info("{} Spring proxies reset", i);
+            LOGGER.debug("{} Spring proxies reset", i);
         } else {
             LOGGER.debug("No spring proxies reset");
         }
@@ -144,7 +144,7 @@ public Object getBean() throws IllegalAccessException, InvocationTargetException
                         LOGGER.debug("Bean of '{}' not loaded, {} ", bean.getClass().getName(), paramValues);
                         break;
                     }
-                    LOGGER.info("Bean '{}' loaded", bean.getClass().getName());
+                    LOGGER.debug("Bean '{}' loaded", bean.getClass().getName());
                     break;
                 }
             }

diff --git a/...-spring-plugin/src/main/java/org/hotswap/agent/plugin/spring/reload/SpringBeanReload.java b/...-spring-plugin/src/main/java/org/hotswap/agent/plugin/spring/reload/SpringBeanReload.java
@@ -95,9 +95,9 @@ public void addClass(Class clazz) {
         synchronized (classes) {
             if (classes.add(clazz)) {
                 if (sameClass) {
-                    LOGGER.info("try to add changed class '{}' into {}", clazz.getName(), ObjectUtils.identityToString(beanFactory));
+                    LOGGER.debug("try to add changed class '{}' into {}", clazz.getName(), ObjectUtils.identityToString(beanFactory));
                 } else {
-                    LOGGER.info("try to add changed class '{}({})' into {}", clazz.getName(), userClassSimpleName, ObjectUtils.identityToString(beanFactory));
+                    LOGGER.debug("try to add changed class '{}({})' into {}", clazz.getName(), userClassSimpleName, ObjectUtils.identityToString(beanFactory));
                 }
             } else {
                 if (sameClass) {
@@ -252,7 +252,7 @@ public boolean reload(long changeTimeStamps) {
         try {
             origContextClassLoader = ClassUtils.overrideThreadContextClassLoader(beanFactory.getBeanClassLoader());
             beanFactoryAssistant.setReload(true);
-            LOGGER.info("##### start reloading '{}' with timestamp '{}'", ObjectUtils.identityToString(beanFactory), changeTimeStamps);
+            LOGGER.debug("##### start reloading '{}' with timestamp '{}'", ObjectUtils.identityToString(beanFactory), changeTimeStamps);
             LOGGER.trace("SpringReload:{},  beanFactory:{}", this, beanFactory);
             BeanFactoryProcessor.setAllowBeanDefinitionOverriding(beanFactory, true);
             do {
@@ -263,7 +263,7 @@ public boolean reload(long changeTimeStamps) {
             ClassUtils.overrideThreadContextClassLoader(origContextClassLoader);
             beanFactoryAssistant.increaseReloadTimes();
             BeanFactoryProcessor.setAllowBeanDefinitionOverriding(beanFactory, allowBeanDefinitionOverriding);
-            LOGGER.info("##### [{}th] finish reloading '{}', it cost {}ms", beanFactoryAssistant.getReloadTimes(),
+            LOGGER.debug("##### [{}th] finish reloading '{}', it cost {}ms", beanFactoryAssistant.getReloadTimes(),
                     ObjectUtils.identityToString(beanFactory), System.currentTimeMillis() - now);
         }
     }