Fix calculation of maximum instances in CircuitBreakPolicy

chenzhiguo · chenzhiguo · commit dd3ec1d1449a · 2025-04-21T17:29:24.000+08:00
diff --git a/joylive-core/joylive-governance-api/src/main/java/com/jd/live/agent/governance/policy/service/circuitbreak/CircuitBreakPolicy.java b/joylive-core/joylive-governance-api/src/main/java/com/jd/live/agent/governance/policy/service/circuitbreak/CircuitBreakPolicy.java
@@ -382,7 +382,7 @@ public boolean isProtectMode(int instances) {
             return false;
         }
         // The number of instances cannot exceed the maximum limit.
-        int max = (int) Math.ceil(instances * ratio / 100);
+        int max = (int) Math.floor(instances * ratio / 100);
         // The number of instances plus the current request
         int count = inspectors.size() + 1;
         return count > max;

Original file line number	Diff line number	Diff line change
`@@ -382,7 +382,7 @@ public boolean isProtectMode(int instances) {`
`382`	`382`	`return false;`
`383`	`383`	`}`
`384`	`384`	`// The number of instances cannot exceed the maximum limit.`
`385`		`- int max = (int) Math.ceil(instances * ratio / 100);`
	`385`	`+ int max = (int) Math.floor(instances * ratio / 100);`
`386`	`386`	`// The number of instances plus the current request`
`387`	`387`	`int count = inspectors.size() + 1;`
`388`	`388`	`return count > max;`