You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dubbo.apache.org by al...@apache.org on 2022/09/15 06:27:51 UTC

[dubbo] branch 3.1 updated: fix: Fix unregisterServiceInstance not work in shutdown hook (#10610) (#10613)

This is an automated email from the ASF dual-hosted git repository.

albumenj pushed a commit to branch 3.1
in repository https://gitbox.apache.org/repos/asf/dubbo.git


The following commit(s) were added to refs/heads/3.1 by this push:
     new 96258e5245 fix: Fix unregisterServiceInstance not work in shutdown hook (#10610) (#10613)
96258e5245 is described below

commit 96258e52450814061b7d0caa58925bf632f56354
Author: xh1202 <38...@users.noreply.github.com>
AuthorDate: Thu Sep 15 14:27:45 2022 +0800

    fix: Fix unregisterServiceInstance not work in shutdown hook (#10610) (#10613)
---
 .../org/apache/dubbo/config/deploy/DefaultApplicationDeployer.java     | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/deploy/DefaultApplicationDeployer.java b/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/deploy/DefaultApplicationDeployer.java
index ac3e0212af..c5e19c4bd0 100644
--- a/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/deploy/DefaultApplicationDeployer.java
+++ b/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/deploy/DefaultApplicationDeployer.java
@@ -764,15 +764,14 @@ public class DefaultApplicationDeployer extends AbstractDeployer<ApplicationMode
             }
             onStopping();
 
+            unregisterServiceInstance();
             destroyRegistries();
-            destroyServiceDiscoveries();
             destroyMetadataReports();
 
             unRegisterShutdownHook();
             if (asyncMetadataFuture != null) {
                 asyncMetadataFuture.cancel(true);
             }
-            unregisterServiceInstance();
         }
     }