svn commit: r1483445 - in /incubator/ambari/trunk: ./ ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/ ambari-server/src/test/java/org/apache/ambari/server/controller/ganglia/

[swagle]

Author: swagle
Date: Thu May 16 17:00:26 2013
New Revision: 1483445

URL: http://svn.apache.org/r1483445
Log:
AMBARI-2107. Cluster CPU Chart is off the charts. Reopened. (swagle)

Modified:
    incubator/ambari/trunk/CHANGES.txt
    
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaMetric.java
    
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
    
incubator/ambari/trunk/ambari-server/src/test/java/org/apache/ambari/server/controller/ganglia/GangliaMetricTest.java

Modified: incubator/ambari/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/CHANGES.txt?rev=1483445&r1=1483444&r2=1483445&view=diff
==============================================================================
--- incubator/ambari/trunk/CHANGES.txt (original)
+++ incubator/ambari/trunk/CHANGES.txt Thu May 16 17:00:26 2013
@@ -854,6 +854,8 @@ Trunk (unreleased changes):
 
  BUG FIXES
 
+ AMBARI-2107. Cluster CPU Chart is off the charts. Reopened. (swagle)
+
  AMBARI-2139. Hive Service check fails on secure cluster. (jaimin)
 
  AMBARI-2138. HBase smoke test fails on secure cluster. (swagle)

Modified: 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaMetric.java
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaMetric.java?rev=1483445&r1=1483444&r2=1483445&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaMetric.java
 (original)
+++ 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaMetric.java
 Thu May 16 17:00:26 2013
@@ -71,11 +71,11 @@ public class GangliaMetric {
   static {
     Set<String> temp = new HashSet<String>();
     temp.add("cpu_wio");
-    /*temp.add("cpu_idle");
+    temp.add("cpu_idle");
     temp.add("cpu_nice");
     temp.add("cpu_aidle");
     temp.add("cpu_system");
-    temp.add("cpu_user");*/
+    temp.add("cpu_user");
     PERCENTAGE_METRIC = Collections.unmodifiableSet(temp);
   }
 
@@ -214,18 +214,18 @@ public class GangliaMetric {
   public static class TemporalMetric {
     private Number m_value;
     private Number m_time;
-    private boolean isInvalid;
+    private boolean valid;
 
-    public boolean isIsInvalid() {
-      return isInvalid;
+    public boolean isValid() {
+      return valid;
     }
 
     public TemporalMetric(String value, Number time) {
-      isInvalid = false;
+      valid = true;
       try{
         m_value = convertToNumber(value);
       } catch (NumberFormatException e) {
-        isInvalid = true;
+        valid = false;
       }
       m_time = time;
     }

Modified: 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java?rev=1483445&r1=1483444&r2=1483445&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
 (original)
+++ 
incubator/ambari/trunk/ambari-server/src/main/java/org/apache/ambari/server/controller/ganglia/GangliaPropertyProvider.java
 Thu May 16 17:00:26 2013
@@ -414,7 +414,7 @@ public abstract class GangliaPropertyPro
           String val = reader.readLine();
           while(! val.equals("[AMBARI_DP_END]")) {
             GangliaMetric.TemporalMetric tm = new 
GangliaMetric.TemporalMetric(val, time);
-            if (!tm.isIsInvalid()) listTemporalMetrics.add(tm);
+            if (tm.isValid()) listTemporalMetrics.add(tm);
             time += step;
             val = reader.readLine();
           }

Modified: 
incubator/ambari/trunk/ambari-server/src/test/java/org/apache/ambari/server/controller/ganglia/GangliaMetricTest.java
URL: 
http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-server/src/test/java/org/apache/ambari/server/controller/ganglia/GangliaMetricTest.java?rev=1483445&r1=1483444&r2=1483445&view=diff
==============================================================================
--- 
incubator/ambari/trunk/ambari-server/src/test/java/org/apache/ambari/server/controller/ganglia/GangliaMetricTest.java
 (original)
+++ 
incubator/ambari/trunk/ambari-server/src/test/java/org/apache/ambari/server/controller/ganglia/GangliaMetricTest.java
 Thu May 16 17:00:26 2013
@@ -97,7 +97,7 @@ public class GangliaMetricTest {
     System.out.println("GangliaMetric.TemporalMetric");
     GangliaMetric.TemporalMetric tm;
     tm = new GangliaMetric.TemporalMetric("100", new Long(1362440880));
-    assertFalse("GangliaMetric.TemporalMetric is valid", tm.isIsInvalid());
+    assertTrue("GangliaMetric.TemporalMetric is valid", tm.isValid());
   }
 
     /**
@@ -108,7 +108,7 @@ public class GangliaMetricTest {
     System.out.println("GangliaMetric.TemporalMetric");
     GangliaMetric.TemporalMetric tm;
     tm = new GangliaMetric.TemporalMetric("any string", new Long(1362440880));
-    assertTrue("GangliaMetric.TemporalMetric is invalid", tm.isIsInvalid());
+    assertFalse("GangliaMetric.TemporalMetric is invalid", tm.isValid());
   }