You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by mp...@apache.org on 2017/04/28 03:16:12 UTC
[2/3] mesos git commit: Removed an incorrect `CHECK` in the master.
Removed an incorrect `CHECK` in the master.
Review: https://reviews.apache.org/r/58688/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/6ace0a76
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/6ace0a76
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/6ace0a76
Branch: refs/heads/master
Commit: 6ace0a762a97a87f0fa28adc6072aa8b039360c3
Parents: 88e2189
Author: Michael Park <mp...@apache.org>
Authored: Thu Apr 27 20:14:58 2017 -0700
Committer: Michael Park <mp...@apache.org>
Committed: Thu Apr 27 20:14:58 2017 -0700
----------------------------------------------------------------------
src/master/master.cpp | 2 --
1 file changed, 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/6ace0a76/src/master/master.cpp
----------------------------------------------------------------------
diff --git a/src/master/master.cpp b/src/master/master.cpp
index a0cb505..e8c2a96 100644
--- a/src/master/master.cpp
+++ b/src/master/master.cpp
@@ -399,8 +399,6 @@ void Framework::trackUnderRole(const string& role)
CHECK(!isTrackedUnderRole(role));
- CHECK(roles.count(role) > 0);
-
if (!master->roles.contains(role)) {
master->roles[role] = new Role(role);
}