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

liubao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git


The following commit(s) were added to refs/heads/master by this push:
     new 83585d5e9 [#4593]Fix when API change edge service not load latest 
schema problem (#4594)
83585d5e9 is described below

commit 83585d5e9cb636c0d988f3838457adc305c1d18b
Author: liubao68 <[email protected]>
AuthorDate: Fri Nov 8 09:01:28 2024 +0800

    [#4593]Fix when API change edge service not load latest schema problem 
(#4594)
---
 .../java/org/apache/servicecomb/registry/DiscoveryManager.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/DiscoveryManager.java
 
b/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/DiscoveryManager.java
index 256b392de..2cbe83a24 100644
--- 
a/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/DiscoveryManager.java
+++ 
b/foundations/foundation-registry/src/main/java/org/apache/servicecomb/registry/DiscoveryManager.java
@@ -93,10 +93,6 @@ public class DiscoveryManager implements LifeCycle {
 
   private void onInstancesChanged(String registryName, String application, 
String serviceName,
       List<? extends DiscoveryInstance> instances) {
-    for (InstanceChangeListener listener : this.instanceChangeListeners) {
-      listener.onInstancesChanged(registryName, application, serviceName, 
instances);
-    }
-
     Map<String, StatefulDiscoveryInstance> statefulInstances = 
allInstances.computeIfAbsent(application, key ->
         new ConcurrentHashMapEx<>()).computeIfAbsent(serviceName, key -> new 
ConcurrentHashMapEx<>());
 
@@ -136,6 +132,10 @@ public class DiscoveryManager implements LifeCycle {
         application, serviceName, instances.size(), instanceInfo);
 
     rebuildVersionCache(application, serviceName);
+
+    for (InstanceChangeListener listener : this.instanceChangeListeners) {
+      listener.onInstancesChanged(registryName, application, serviceName, 
instances);
+    }
   }
 
   public void addInstanceChangeListener(InstanceChangeListener 
instanceChangeListener) {

Reply via email to