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

dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new b6b7eab8d48 [fix](cloud) Fix cloud fe cluster metrics not registered 
in FOLLOWER (#51377)
b6b7eab8d48 is described below

commit b6b7eab8d48dc78ea5b747d15bdd8e68fbceed78
Author: deardeng <[email protected]>
AuthorDate: Mon Jun 2 10:37:02 2025 +0800

    [fix](cloud) Fix cloud fe cluster metrics not registered in FOLLOWER 
(#51377)
---
 .../main/java/org/apache/doris/cloud/catalog/CloudClusterChecker.java   | 1 -
 .../main/java/org/apache/doris/cloud/system/CloudSystemInfoService.java | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudClusterChecker.java
 
b/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudClusterChecker.java
index fb0803bed17..49198536444 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudClusterChecker.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudClusterChecker.java
@@ -106,7 +106,6 @@ public class CloudClusterChecker extends MasterDaemon {
                 // For old versions that do no have status field set
                 ClusterStatus clusterStatus = 
remoteClusterIdToPB.get(addId).hasClusterStatus()
                         ? remoteClusterIdToPB.get(addId).getClusterStatus() : 
ClusterStatus.NORMAL;
-                MetricRepo.registerCloudMetrics(clusterId, clusterName);
                 List<Backend> toAdd = new ArrayList<>();
                 for (Cloud.NodeInfoPB node : 
remoteClusterIdToPB.get(addId).getNodesList()) {
                     String addr = Config.enable_fqdn_mode ? node.getHost() : 
node.getIp();
diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/cloud/system/CloudSystemInfoService.java
 
b/fe/fe-core/src/main/java/org/apache/doris/cloud/system/CloudSystemInfoService.java
index 2d698a5fc86..7c8af713fb1 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/cloud/system/CloudSystemInfoService.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/cloud/system/CloudSystemInfoService.java
@@ -236,7 +236,6 @@ public class CloudSystemInfoService extends 
SystemInfoService {
             if (be == null) {
                 be = new ArrayList<>();
                 clusterIdToBackend.put(clusterId, be);
-                MetricRepo.registerCloudMetrics(clusterId, clusterName);
             }
             Set<String> existed = be.stream().map(i -> i.getHost() + ":" + 
i.getHeartbeatPort())
                     .collect(Collectors.toSet());
@@ -251,6 +250,7 @@ public class CloudSystemInfoService extends 
SystemInfoService {
             sortBackends.add(b);
             Collections.sort(sortBackends, 
Comparator.comparing(Backend::getId));
             clusterIdToBackend.put(clusterId, sortBackends);
+            MetricRepo.registerCloudMetrics(clusterId, clusterName);
             LOG.info("update (add) cloud cluster map, clusterName={} 
clusterId={} backendNum={} current backend={}",
                     clusterName, clusterId, sortBackends.size(), sortBackends);
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to