Merge pull request #1407 from mattrjacobs/reset-percentile-snapshot-when-advancing-all-buckets

mattrjacobs · web-flow · commit b0f5152f2215 · 2016-10-26T20:46:28.000-07:00
Clear the percentile snapshot whenever all buckets get cleared
diff --git a/hystrix-core/src/main/java/com/netflix/hystrix/util/HystrixRollingPercentile.java b/hystrix-core/src/main/java/com/netflix/hystrix/util/HystrixRollingPercentile.java
@@ -308,6 +308,9 @@ public void reset() {
 
         // clear buckets so we start over again
         buckets.clear();
+
+        // and also make sure the percentile snapshot gets reset
+        currentPercentileSnapshot = new PercentileSnapshot(buckets.getArray());
     }
 
     /* package-private for testing */ static class PercentileBucketData {

Original file line number	Diff line number	Diff line change
`@@ -308,6 +308,9 @@ public void reset() {`
`308`	`308`
`309`	`309`	`// clear buckets so we start over again`
`310`	`310`	`buckets.clear();`
	`311`	`+`
	`312`	`+ // and also make sure the percentile snapshot gets reset`
	`313`	`+ currentPercentileSnapshot = new PercentileSnapshot(buckets.getArray());`
`311`	`314`	`}`
`312`	`315`
`313`	`316`	`/* package-private for testing */ static class PercentileBucketData {`