You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Vinod Kone (JIRA)" <ji...@apache.org> on 2015/09/01 02:06:46 UTC
[jira] [Commented] (MESOS-3301) Should remove redundant check
[ https://issues.apache.org/jira/browse/MESOS-3301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14724437#comment-14724437 ]
Vinod Kone commented on MESOS-3301:
-----------------------------------
Commented on the ticket.
As a piece of advice, I would highly recommend looking for a shepherd and discussing with him/her, *before* sending out reviews. It will help streamline the review process and avoid delays.
Looking forward to your contributions!
> Should remove redundant check
> -----------------------------
>
> Key: MESOS-3301
> URL: https://issues.apache.org/jira/browse/MESOS-3301
> Project: Mesos
> Issue Type: Improvement
> Reporter: Yong Qiao Wang
> Assignee: Yong Qiao Wang
> Priority: Minor
>
> In functions: HierarchicalAllocatorProcess<RoleSorter, FrameworkSorter>::initialize() and HierarchicalAllocatorProcess<RoleSorter, FrameworkSorter>::allocate(), it does not need to check the roleSorter->count() , because the role * always be added to roleSorter.
> So suggest to remove the following lines from initialize() and allocate() functions:
> -if (roleSorter->count() == 0) {
> - LOG(ERROR) << "No roles specified, cannot allocate resources!";
> - return;
> -}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)