(incubator-uniffle) branch master updated: [MINOR] fix(coordinator): Correcting Assignment of appCurrent and appTotality (#2050)

[zuston]

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

zuston pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new db1424152 [MINOR] fix(coordinator): Correcting Assignment of 
appCurrent and appTotality (#2050)
db1424152 is described below

commit db1424152b7f7262f338d469368ad4dd5881b4d8
Author: maobaolong <baoloong...@tencent.com>
AuthorDate: Mon Aug 19 11:08:28 2024 +0800

    [MINOR] fix(coordinator): Correcting Assignment of appCurrent and 
appTotality (#2050)
    
    ### What changes were proposed in this pull request?
    
    Correcting Assignment of appCurrent and appTotality
    
    ### Why are the changes needed?
    
    The `appCurrent` stand for the running app count.
    
    The `appTotality` stand for the total app count include running and 
finished.
    
    ### Does this PR introduce _any_ user-facing change?
    
    (Please list the user-facing changes introduced by your change, including
      1. Change in user-facing APIs.
      2. Addition or removal of property keys.)
    
    `APPS TOTAL` and `APPS CURRENT` swapped.
    
    ### How was this patch tested?
    
    No need
---
 .../apache/uniffle/coordinator/web/resource/ApplicationResource.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/web/resource/ApplicationResource.java
 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/web/resource/ApplicationResource.java
index 9839ec92e..eaa7695d3 100644
--- 
a/coordinator/src/main/java/org/apache/uniffle/coordinator/web/resource/ApplicationResource.java
+++ 
b/coordinator/src/main/java/org/apache/uniffle/coordinator/web/resource/ApplicationResource.java
@@ -51,8 +51,8 @@ public class ApplicationResource extends BaseResource {
     return execute(
         () -> {
           Map<String, Integer> appTotalityMap = Maps.newHashMap();
-          appTotalityMap.put("appTotality", 
getApplicationManager().getAppIds().size());
-          appTotalityMap.put("appCurrent", (int) 
CoordinatorMetrics.counterTotalAppNum.get());
+          appTotalityMap.put("appCurrent", 
getApplicationManager().getAppIds().size());
+          appTotalityMap.put("appTotality", (int) 
CoordinatorMetrics.counterTotalAppNum.get());
           return appTotalityMap;
         });
   }