#4900 Fix linkismanager allocation ECM error logic, adjust resource a…

…llocation logic from large to small (#4901) Co-authored-by: weipengfei <[email protected]>
apache · Sep 8, 2023 · 8dfcf86 · 8dfcf86
1 parent 8181efc
commit 8dfcf86
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...ager/src/main/java/org/apache/linkis/manager/am/selector/rule/ResourceNodeSelectRule.java b/...ager/src/main/java/org/apache/linkis/manager/am/selector/rule/ResourceNodeSelectRule.java
@@ -67,9 +67,9 @@ private Comparator<Node> sortByResource() {
                 .getNodeResource()
                 .getLeftResource()
                 .moreThan(nodeBRm.getNodeResource().getLeftResource())) {
-              return 1;
-            } else {
               return -1;
+            } else {
+              return 1;
             }
           }
         } catch (Throwable t) {

diff --git a/...manager/src/main/java/org/apache/linkis/manager/am/selector/rule/ScoreNodeSelectRule.java b/...manager/src/main/java/org/apache/linkis/manager/am/selector/rule/ScoreNodeSelectRule.java
@@ -55,13 +55,13 @@ private Comparator<Node> sortByScore() {
         ScoreServiceInstance instanceB = (ScoreServiceInstance) nodeB;
         try {
           if (instanceA.getScore() > instanceB.getScore()) {
-            return 1;
+            return -1;
           }
         } catch (Exception e) {
           logger.warn("Failed to Compare resource ", e);
           return -1;
         }
-        return -1;
+        return 1;
       } else {
         return -1;
       }