You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ry...@apache.org on 2012/04/18 09:52:25 UTC

svn commit: r1327410 - /lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/admin/SolrInfoMBeanHandler.java

Author: ryan
Date: Wed Apr 18 07:52:24 2012
New Revision: 1327410

URL: http://svn.apache.org/viewvc?rev=1327410&view=rev
Log:
SOLR-3373: more careful NaN diff calculations

Modified:
    lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/admin/SolrInfoMBeanHandler.java

Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/admin/SolrInfoMBeanHandler.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/admin/SolrInfoMBeanHandler.java?rev=1327410&r1=1327409&r2=1327410&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/admin/SolrInfoMBeanHandler.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/handler/admin/SolrInfoMBeanHandler.java Wed Apr 18 07:52:24 2012
@@ -246,7 +246,18 @@ public class SolrInfoMBeanHandler extend
       if((ref instanceof Double) || (ref instanceof Float)) {
         nf = NumberFormat.getInstance(Locale.ENGLISH);
       }
-      double diff = ((Number)now).doubleValue() - ((Number)ref).doubleValue();
+      double dref = ((Number)ref).doubleValue();
+      double dnow = ((Number)now).doubleValue();
+      double diff = Double.NaN;
+      if(Double.isNaN(dref)) {
+        diff = dnow;
+      }
+      else if(Double.isNaN(dnow)) {
+        diff = dref;
+      }
+      else {
+        diff = dnow-dref;
+      }
       str.append( ", Delta: ").append(nf.format(diff));
     }
     return str.toString();