You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2018/09/06 02:00:30 UTC

[mesos] 01/03: Avoided double-lookup in Master::getFramework.

This is an automated email from the ASF dual-hosted git repository.

bmahler pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit 804b4a59b17a0749f80cb2010630d7907a148683
Author: Benjamin Mahler <bm...@apache.org>
AuthorDate: Wed Sep 5 14:49:40 2018 -0700

    Avoided double-lookup in Master::getFramework.
    
    Review: https://reviews.apache.org/r/68634
---
 src/master/master.cpp | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/master/master.cpp b/src/master/master.cpp
index e307df0..7bc924f 100644
--- a/src/master/master.cpp
+++ b/src/master/master.cpp
@@ -11476,9 +11476,7 @@ bool Master::isCompletedFramework(const FrameworkID& frameworkId)
 // TODO(bmahler): Consider killing this.
 Framework* Master::getFramework(const FrameworkID& frameworkId) const
 {
-  return frameworks.registered.contains(frameworkId)
-           ? frameworks.registered.at(frameworkId)
-           : nullptr;
+  return frameworks.registered.get(frameworkId).getOrElse(nullptr);
 }