filter out crappy gc reporting

0xPIT · Jun 24, 2011 · 89a46d1 · 89a46d1
1 parent a9a8095
commit 89a46d1
Showing 1 changed file with 4 additions and 0 deletions.
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
@@ -108,6 +108,10 @@ private void monitorLongGc() {
                         // 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()) {