From 51eadd43c47c043cb1d06ea3f437020bf4feec3a Mon Sep 17 00:00:00 2001 From: Jon Bratseth Date: Mon, 8 Mar 2021 19:51:10 +0100 Subject: Revert "Check for zero minutes" --- .../com/yahoo/vespa/hosted/provision/autoscale/ClusterTimeseries.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/autoscale/ClusterTimeseries.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/autoscale/ClusterTimeseries.java index a5e21c29fb3..150958835ac 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/autoscale/ClusterTimeseries.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/autoscale/ClusterTimeseries.java @@ -55,7 +55,7 @@ public class ClusterTimeseries { for (int start = 0; start < snapshots.size(); start++) { if (start > 0) { // Optimization: Skip this point when starting from the previous is better relative to the best rate so far Duration duration = durationBetween(start - 1, start); - if ( duration.toMinutes() != 0) { + if ( ! duration.isZero()) { double growthRate = (queryRateAt(start - 1) - queryRateAt(start)) / duration.toMinutes(); if (growthRate >= maxGrowthRate) continue; @@ -64,7 +64,7 @@ public class ClusterTimeseries { for (int end = start + 1; end < snapshots.size(); end++) { if (queryRateAt(end) >= queryRateAt(start) * 1.3) { Duration duration = durationBetween(start, end); - if (duration.toMinutes() == 0) continue; + if (duration.isZero()) continue; double growthRate = (queryRateAt(end) - queryRateAt(start)) / duration.toMinutes(); if (growthRate > maxGrowthRate) maxGrowthRate = growthRate; -- cgit v1.2.3