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

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


The following commit(s) were added to refs/heads/3.2 by this push:
     new 6ded7b23ff Fix hasPreparedInternalModule is always false. (#11535) 
(#11536)
6ded7b23ff is described below

commit 6ded7b23ffdd95cdf9b9bab27ed69ef57d0e4743
Author: KamTo Hung <[email protected]>
AuthorDate: Tue Feb 14 10:49:31 2023 +0800

    Fix hasPreparedInternalModule is always false. (#11535) (#11536)
    
    * Fix hasPreparedInternalModule is always false. (#11535)
    
    * Fix hasPreparedInternalModule is always false. (#11535)
---
 .../org/apache/dubbo/config/deploy/DefaultApplicationDeployer.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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 bee56527c2..f1174c5759 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
@@ -714,6 +714,7 @@ public class DefaultApplicationDeployer extends 
AbstractDeployer<ApplicationMode
                 // wait for internal module startup
                 try {
                     future.get(5, TimeUnit.SECONDS);
+                    hasPreparedInternalModule = true;
                 } catch (Exception e) {
                     logger.warn(CONFIG_FAILED_START_MODEL, "", "", "wait for 
internal module startup failed: " + e.getMessage(), e);
                 }
@@ -770,7 +771,7 @@ public class DefaultApplicationDeployer extends 
AbstractDeployer<ApplicationMode
                 dynamicConfiguration = 
getDynamicConfiguration(configCenter.toUrl());
             } catch (Exception e) {
                 if (!configCenter.isCheck()) {
-                    logger.warn(CONFIG_FAILED_INIT_CONFIG_CENTER, "", "","The 
configuration center failed to initialize", e);
+                    logger.warn(CONFIG_FAILED_INIT_CONFIG_CENTER, "", "", "The 
configuration center failed to initialize", e);
                     configCenter.setInitialized(false);
                     return null;
                 } else {

Reply via email to