From 89a46d1c875bd167bd3c0ff369aa4c89eca9cdf3 Mon Sep 17 00:00:00 2001
From: kimchy <kimchy@gmail.com>
Date: Fri, 24 Jun 2011 13:18:16 +0300
Subject: [PATCH] filter out crappy gc reporting

---
 .../java/org/elasticsearch/monitor/jvm/JvmMonitorService.java | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/modules/elasticsearch/src/main/java/org/elasticsearch/monitor/jvm/JvmMonitorService.java b/modules/elasticsearch/src/main/java/org/elasticsearch/monitor/jvm/JvmMonitorService.java
index e8d5c2a1826..9ff254f970e 100644
--- a/modules/elasticsearch/src/main/java/org/elasticsearch/monitor/jvm/JvmMonitorService.java
+++ b/modules/elasticsearch/src/main/java/org/elasticsearch/monitor/jvm/JvmMonitorService.java
@@ -108,6 +108,10 @@ public class JvmMonitorService extends AbstractLifecycleComponent<JvmMonitorServ
                         // we already handled this one...
                         continue;
                     }
+                    // Ignore any duration > 1hr; getLastGcInfo occasionally returns total crap
+                    if (lastGc.duration().hoursFrac() > 1) {
+                        continue;
+                    }
                     if (lastGc.duration().millis() > gcThreshold.millis()) {
                         logger.info("[gc][{}][{}] took [{}]/[{}], reclaimed [{}], leaving [{}] used, max [{}]", gc.name(), gc.getCollectionCount(), lastGc.duration(), gc.getCollectionTime(), lastGc.reclaimed(), lastGc.afterUsed(), lastGc.max());
                     } else if (logger.isDebugEnabled()) {