update destruction log and give better messages

alibaba · Jan 2, 2025 · 90a118a · 90a118a
1 parent 14becb8
commit 90a118a
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 14 deletions.
diff --git a/client/src/main/java/com/alibaba/nacos/client/config/impl/ConfigHttpClientManager.java b/client/src/main/java/com/alibaba/nacos/client/config/impl/ConfigHttpClientManager.java
@@ -71,14 +71,14 @@ public static ConfigHttpClientManager getInstance() {
 
     @Override
     public void shutdown() throws NacosException {
-        NAMING_LOGGER.warn("[ConfigHttpClientManager] Start destroying NacosRestTemplate");
+        NAMING_LOGGER.info("[ConfigHttpClientManager] Start destroying NacosRestTemplate");
         try {
             HttpClientBeanHolder.shutdownNacosSyncRest(HTTP_CLIENT_FACTORY.getClass().getName());
         } catch (Exception ex) {
             NAMING_LOGGER.error("[ConfigHttpClientManager] An exception occurred when the HTTP client was closed : {}",
                     ExceptionUtil.getStackTrace(ex));
         }
-        NAMING_LOGGER.warn("[ConfigHttpClientManager] Destruction of the end");
+        NAMING_LOGGER.info("[ConfigHttpClientManager] Completed destruction of NacosRestTemplate");
     }
 
     /**

diff --git a/...nt/src/main/java/com/alibaba/nacos/client/naming/remote/http/NamingHttpClientManager.java b/...nt/src/main/java/com/alibaba/nacos/client/naming/remote/http/NamingHttpClientManager.java
@@ -68,14 +68,14 @@ public NacosRestTemplate getNacosRestTemplate() {
 
     @Override
     public void shutdown() throws NacosException {
-        NAMING_LOGGER.warn("[NamingHttpClientManager] Start destroying NacosRestTemplate");
+        NAMING_LOGGER.info("[NamingHttpClientManager] Start destroying NacosRestTemplate");
         try {
             HttpClientBeanHolder.shutdownNacosSyncRest(HTTP_CLIENT_FACTORY.getClass().getName());
         } catch (Exception ex) {
             NAMING_LOGGER.error("[NamingHttpClientManager] An exception occurred when the HTTP client was closed : {}",
                     ExceptionUtil.getStackTrace(ex));
         }
-        NAMING_LOGGER.warn("[NamingHttpClientManager] Destruction of the end");
+        NAMING_LOGGER.info("[NamingHttpClientManager] Completed destruction of NacosRestTemplate");
     }
 
     private static class NamingHttpClientFactory extends AbstractHttpClientFactory {

diff --git a/common/src/main/java/com/alibaba/nacos/common/executor/ThreadPoolManager.java b/common/src/main/java/com/alibaba/nacos/common/executor/ThreadPoolManager.java
@@ -51,9 +51,9 @@ public final class ThreadPoolManager {
     static {
         INSTANCE.init();
         ThreadUtils.addShutdownHook(new Thread(() -> {
-            LOGGER.warn("[ThreadPoolManager] Start destroying ThreadPool");
+            LOGGER.info("[ThreadPoolManager] Start destroying ThreadPool");
             shutdown();
-            LOGGER.warn("[ThreadPoolManager] Destruction of the end");
+            LOGGER.info("[ThreadPoolManager] Completed destruction of ThreadPool");
         }));
     }
 

diff --git a/common/src/main/java/com/alibaba/nacos/common/http/HttpClientBeanHolder.java b/common/src/main/java/com/alibaba/nacos/common/http/HttpClientBeanHolder.java
@@ -99,7 +99,7 @@ private static void shutdown() {
         if (!ALREADY_SHUTDOWN.compareAndSet(false, true)) {
             return;
         }
-        LOGGER.warn("[HttpClientBeanHolder] Start destroying common HttpClient");
+        LOGGER.info("[HttpClientBeanHolder] Start destroying common HttpClient");
 
         try {
             shutdown(DefaultHttpClientFactory.class.getName());
@@ -108,7 +108,7 @@ private static void shutdown() {
                     ExceptionUtil.getStackTrace(ex));
         }
 
-        LOGGER.warn("[HttpClientBeanHolder] Destruction of the end");
+        LOGGER.info("[HttpClientBeanHolder] Completed destruction of HttpClient");
     }
 
     /**

diff --git a/common/src/main/java/com/alibaba/nacos/common/notify/NotifyCenter.java b/common/src/main/java/com/alibaba/nacos/common/notify/NotifyCenter.java
@@ -131,7 +131,7 @@ public static void shutdown() {
         if (!CLOSED.compareAndSet(false, true)) {
             return;
         }
-        LOGGER.warn("[NotifyCenter] Start destroying Publisher");
+        LOGGER.info("[NotifyCenter] Start destroying Publisher");
 
         for (Map.Entry<String, EventPublisher> entry : INSTANCE.publisherMap.entrySet()) {
             try {
@@ -148,7 +148,7 @@ public static void shutdown() {
             LOGGER.error("[SharePublisher] shutdown has error : ", e);
         }
 
-        LOGGER.warn("[NotifyCenter] Destruction of the end");
+        LOGGER.info("[NotifyCenter] Completed destruction of Publisher");
     }
 
     /**

diff --git a/config/src/main/java/com/alibaba/nacos/config/server/service/notify/HttpClientManager.java b/config/src/main/java/com/alibaba/nacos/config/server/service/notify/HttpClientManager.java
@@ -64,7 +64,7 @@ public static NacosAsyncRestTemplate getNacosAsyncRestTemplate() {
     }
 
     private static void shutdown() {
-        LOGGER.warn("[ConfigServer-HttpClientManager] Start destroying NacosRestTemplate");
+        LOGGER.info("[ConfigServer-HttpClientManager] Start destroying NacosRestTemplate");
         try {
             final String httpClientFactoryBeanName = ConfigHttpClientFactory.class.getName();
             HttpClientBeanHolder.shutdownNacosSyncRest(httpClientFactoryBeanName);
@@ -73,7 +73,7 @@ private static void shutdown() {
             LOGGER.error("[ConfigServer-HttpClientManager] An exception occurred when the HTTP client was closed : {}",
                     ExceptionUtil.getStackTrace(ex));
         }
-        LOGGER.warn("[ConfigServer-HttpClientManager] Destruction of the end");
+        LOGGER.info("[ConfigServer-HttpClientManager] Completed destruction of NacosRestTemplate");
     }
 
     /**

diff --git a/naming/src/main/java/com/alibaba/nacos/naming/misc/HttpClientManager.java b/naming/src/main/java/com/alibaba/nacos/naming/misc/HttpClientManager.java
@@ -104,7 +104,7 @@ public static NacosAsyncRestTemplate getProcessorNacosAsyncRestTemplate() {
     }
 
     private static void shutdown() {
-        SRV_LOG.warn("[NamingServerHttpClientManager] Start destroying HTTP-Client");
+        SRV_LOG.info("[NamingServerHttpClientManager] Start destroying HTTP-Client");
         try {
             HttpClientBeanHolder.shutdownNacosSyncRest(SYNC_HTTP_CLIENT_FACTORY.getClass().getName());
             HttpClientBeanHolder.shutdownNacosSyncRest(APACHE_SYNC_HTTP_CLIENT_FACTORY.getClass().getName());
@@ -114,7 +114,7 @@ private static void shutdown() {
             SRV_LOG.error("[NamingServerHttpClientManager] An exception occurred when the HTTP client was closed : {}",
                     ExceptionUtil.getStackTrace(ex));
         }
-        SRV_LOG.warn("[NamingServerHttpClientManager] Destruction of the end");
+        SRV_LOG.info("[NamingServerHttpClientManager] Completed destruction of HTTP-Client");
     }
 
     private static class AsyncHttpClientFactory extends AbstractHttpClientFactory {