You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by jo...@apache.org on 2015/09/17 00:43:59 UTC

[2/5] mesos git commit: Revert "Integer Precision for JSON <-> Protobuf conversions."

Revert "Integer Precision for JSON <-> Protobuf conversions."

This reverts commit df9eacb10fbec7758bde1b05f35661e2e82441f4.


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/fa80dcb5
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/fa80dcb5
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/fa80dcb5

Branch: refs/heads/master
Commit: fa80dcb59d436e5679b093e0bdda2b72b7b644bd
Parents: ceb832b
Author: Joris Van Remoortere <jo...@gmail.com>
Authored: Wed Sep 16 18:42:41 2015 -0400
Committer: Joris Van Remoortere <jo...@gmail.com>
Committed: Wed Sep 16 18:42:41 2015 -0400

----------------------------------------------------------------------
 3rdparty/libprocess/src/tests/metrics_tests.cpp | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/fa80dcb5/3rdparty/libprocess/src/tests/metrics_tests.cpp
----------------------------------------------------------------------
diff --git a/3rdparty/libprocess/src/tests/metrics_tests.cpp b/3rdparty/libprocess/src/tests/metrics_tests.cpp
index 8f15ee7..29ed033 100644
--- a/3rdparty/libprocess/src/tests/metrics_tests.cpp
+++ b/3rdparty/libprocess/src/tests/metrics_tests.cpp
@@ -193,10 +193,10 @@ TEST(MetricsTest, Snapshot)
   map<string, JSON::Value> values = responseJSON.get().values;
 
   EXPECT_EQ(1u, values.count("test/counter"));
-  EXPECT_FLOAT_EQ(0.0, values["test/counter"].as<JSON::Number>().as<double>());
+  EXPECT_FLOAT_EQ(0.0, values["test/counter"].as<JSON::Number>().value);
 
   EXPECT_EQ(1u, values.count("test/gauge"));
-  EXPECT_FLOAT_EQ(42.0, values["test/gauge"].as<JSON::Number>().as<double>());
+  EXPECT_FLOAT_EQ(42.0, values["test/gauge"].as<JSON::Number>().value);
 
   EXPECT_EQ(0u, values.count("test/gauge_fail"));
 
@@ -290,10 +290,10 @@ TEST(MetricsTest, SnapshotTimeout)
   map<string, JSON::Value> values = responseJSON.get().values;
 
   EXPECT_EQ(1u, values.count("test/counter"));
-  EXPECT_FLOAT_EQ(0.0, values["test/counter"].as<JSON::Number>().as<double>());
+  EXPECT_FLOAT_EQ(0.0, values["test/counter"].as<JSON::Number>().value);
 
   EXPECT_EQ(1u, values.count("test/gauge"));
-  EXPECT_FLOAT_EQ(42.0, values["test/gauge"].as<JSON::Number>().as<double>());
+  EXPECT_FLOAT_EQ(42.0, values["test/gauge"].as<JSON::Number>().value);
 
   EXPECT_EQ(0u, values.count("test/gauge_fail"));
   EXPECT_EQ(0u, values.count("test/gauge_timeout"));
@@ -381,9 +381,7 @@ TEST(MetricsTest, SnapshotStatistics)
                const JSON::Value& value,
                responseJSON.get().values) {
     if (value.is<JSON::Number>()) {
-      // "test/counter/count" is an integer, everything else is a double.
-      JSON::Number number = value.as<JSON::Number>();
-      responseValues[key] = number.as<double>();
+      responseValues[key] = value.as<JSON::Number>().value;
     }
   }