Fix the logic of getting maxSuccessQps in StatisticNode (alibaba#1196)

fangwentong · sczyh30 · commit 11f74b27e071 · 2019-12-20T13:57:53.000+08:00
diff --git a/sentinel-core/src/main/java/com/alibaba/csp/sentinel/node/StatisticNode.java b/sentinel-core/src/main/java/com/alibaba/csp/sentinel/node/StatisticNode.java
@@ -213,7 +213,8 @@ public double successQps() {
 
     @Override
     public double maxSuccessQps() {
-        return rollingCounterInSecond.maxSuccess() * rollingCounterInSecond.getSampleCount();
+        return (double) rollingCounterInSecond.maxSuccess() * rollingCounterInSecond.getSampleCount()
+                / rollingCounterInSecond.getWindowIntervalInSec();
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -213,7 +213,8 @@ public double successQps() {`
`213`	`213`
`214`	`214`	`@Override`
`215`	`215`	`public double maxSuccessQps() {`
`216`		`- return rollingCounterInSecond.maxSuccess() * rollingCounterInSecond.getSampleCount();`
	`216`	`+ return (double) rollingCounterInSecond.maxSuccess() * rollingCounterInSecond.getSampleCount()`
	`217`	`+ / rollingCounterInSecond.getWindowIntervalInSec();`
`217`	`218`	`}`
`218`	`219`
`219`	`220`	`@Override`