Skip to content

Commit

Permalink
[MINOR] fix: Change to get coordinator appCurrent value from metrics (a…
Browse files Browse the repository at this point in the history
…pache#2129)

### What changes were proposed in this pull request?

Change to get coordinator appCurrent value from metrics

### Why are the changes needed?

Keep consistent with `appTotality`, and in case of incorrect result returned from `getAppIds`.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

No need.
  • Loading branch information
maobaolong authored Sep 24, 2024
1 parent 7e84a1a commit 3b01684
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ public Response<Map<String, Integer>> getAppTotality() {
return execute(
() -> {
Map<String, Integer> appTotalityMap = Maps.newHashMap();
appTotalityMap.put("appCurrent", getApplicationManager().getAppIds().size());
appTotalityMap.put("appCurrent", (int) CoordinatorMetrics.gaugeRunningAppNum.get());
appTotalityMap.put("appTotality", (int) CoordinatorMetrics.counterTotalAppNum.get());
return appTotalityMap;
});
Expand Down

0 comments on commit 3b01684

Please sign in to comment.