[jira] [Comment Edited] (MESOS-6684) Update addFramework/removeFramework to handle multi-role frameworks

["Jay Guo (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MESOS-6684?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15721528#comment-15721528 ] 

Jay Guo edited comment on MESOS-6684 at 12/6/16 1:53 PM:
---------------------------------------------------------

RR:
https://reviews.apache.org/r/54360 addFramework
https://reviews.apache.org/r/54362 removeFramework
https://reviews.apache.org/r/54361 test for addFramework
https://reviews.apache.org/r/54363 test for removeFramework


was (Author: guoger):
RR:
https://reviews.apache.org/r/54360 addFramework
https://reviews.apache.org/r/54361 test for addFramework
https://reviews.apache.org/r/54362 removeFramework
https://reviews.apache.org/r/54363 test for removeFramework

> Update addFramework/removeFramework to handle multi-role frameworks
> -------------------------------------------------------------------
>
>                 Key: MESOS-6684
>                 URL: https://issues.apache.org/jira/browse/MESOS-6684
>             Project: Mesos
>          Issue Type: Bug
>            Reporter: Guangya Liu
>            Assignee: Jay Guo
>
> The current master add/remove frameworks only handle single role framework, it should be updated to support multi-role frameworks.
> {code}
>  if (!activeRoles.contains(role)) {
>     activeRoles[role] = new Role();
>   }
>   activeRoles[role]->addFramework(framework);
> {code}
> We should update both {{addFramework}} and {{removeFramework}} in master.cpp to be able to map one framework to multiple roles.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)