From 8c7b8afedb994c5d330c067c03d43f4945adcdd0 Mon Sep 17 00:00:00 2001
From: Stephane Nicoll <snicoll@pivotal.io>
Date: Tue, 16 Jun 2015 10:31:24 +0200
Subject: [PATCH] Polish

---
 .../boot/actuate/cache/EhCacheStatisticsProvider.java       | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/cache/EhCacheStatisticsProvider.java b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/cache/EhCacheStatisticsProvider.java
index 98301e36295..6d4ea92afb5 100644
--- a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/cache/EhCacheStatisticsProvider.java
+++ b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/cache/EhCacheStatisticsProvider.java
@@ -37,8 +37,10 @@ public class EhCacheStatisticsProvider implements CacheStatisticsProvider<EhCach
 		statistics.setSize(ehCacheStatistics.getSize());
 		Double hitRatio = ehCacheStatistics.cacheHitRatio();
 		if (!hitRatio.isNaN()) {
-			statistics.setHitRatio(hitRatio > 1 ? 1 : hitRatio);
-			statistics.setMissRatio(hitRatio > 1 ? 0 : 1 - hitRatio);
+			// ratio is calculated 'racily' and can drift marginally above unity, so we cap it here
+			double sanitizedHitRatio = hitRatio > 1 ? 1 : hitRatio;
+			statistics.setHitRatio(sanitizedHitRatio);
+			statistics.setMissRatio(1 - sanitizedHitRatio);
 		}
 		return statistics;
 	}