You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bb...@apache.org on 2018/03/19 15:23:11 UTC

mesos git commit: Removed some redundant `get` calls.

Repository: mesos
Updated Branches:
  refs/heads/master 32093fe19 -> 95bbe784d


Removed some redundant `get` calls.

Review: https://reviews.apache.org/r/66137


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

Branch: refs/heads/master
Commit: 95bbe784da51b3a7eaeb9127e2541ea0b2af07b5
Parents: 32093fe
Author: Benjamin Bannier <bb...@apache.org>
Authored: Mon Mar 19 16:11:38 2018 +0100
Committer: Benjamin Bannier <bb...@apache.org>
Committed: Mon Mar 19 16:12:25 2018 +0100

----------------------------------------------------------------------
 src/slave/slave.cpp                             | 2 +-
 src/tests/persistent_volume_endpoints_tests.cpp | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/95bbe784/src/slave/slave.cpp
----------------------------------------------------------------------
diff --git a/src/slave/slave.cpp b/src/slave/slave.cpp
index 0962ea7..915d4f1 100644
--- a/src/slave/slave.cpp
+++ b/src/slave/slave.cpp
@@ -2804,7 +2804,7 @@ void Slave::__run(
       executor = doLaunchExecutor();
     }
   } else {
-    if (taskGroup.isNone() && task.get().has_command()) {
+    if (taskGroup.isNone() && task->has_command()) {
       // We are dealing with command task; a new command executor will be
       // launched.
       CHECK(executor == nullptr);

http://git-wip-us.apache.org/repos/asf/mesos/blob/95bbe784/src/tests/persistent_volume_endpoints_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/persistent_volume_endpoints_tests.cpp b/src/tests/persistent_volume_endpoints_tests.cpp
index ebdc409..b9dbb96 100644
--- a/src/tests/persistent_volume_endpoints_tests.cpp
+++ b/src/tests/persistent_volume_endpoints_tests.cpp
@@ -2365,7 +2365,7 @@ TEST_F(PersistentVolumeEndpointsTest, SlavesEndpointFullResources)
   JSON::Value offeredValue = slaveObject.values["offered_resources_full"];
   EXPECT_EQ(
       Resources(CHECK_NOTERROR(
-          Resources::fromJSON(expectedOffered.get().as<JSON::Array>()))),
+          Resources::fromJSON(expectedOffered->as<JSON::Array>()))),
       Resources(CHECK_NOTERROR(
           Resources::fromJSON(offeredValue.as<JSON::Array>()))));