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

ASF GitHub Bot commented on STORM-898:
--------------------------------------

Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/921#discussion_r46696433
  
    --- Diff: 
storm-core/src/jvm/backtype/storm/scheduler/resource/RAS_Node.java ---
    @@ -230,6 +241,61 @@ public void freeTopology(String topId, Cluster 
cluster) {
             _topIdToUsedSlots.remove(topId);
         }
     
    +    public void freeMemory(double amount) {
    +        _availMemory += amount;
    +        LOG.debug("freeing {} memory on node {}...avail mem: {}", amount, 
this.getHostname(), _availMemory);
    +        if (_availMemory > this.getTotalMemoryResources()) {
    +            LOG.warn("Freeing more memory than there exists!");
    +        }
    +    }
    +
    +    public void freeCPU(double amount) {
    +        _availCPU += amount;
    +        LOG.debug("freeing {} CPU on node...avail CPU: {}", amount, 
this.getHostname(), _availCPU);
    +        if (_availCPU > this.getAvailableCpuResources()) {
    +            LOG.warn("Freeing more memory than there exists!");
    +        }
    +    }
    --- End diff --
    
    Why are freeMemory and freeCPU public?  They seem like they should be 
private or package private.


> Add priorities and per user resource guarantees to Resource Aware Scheduler
> ---------------------------------------------------------------------------
>
>                 Key: STORM-898
>                 URL: https://issues.apache.org/jira/browse/STORM-898
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>            Reporter: Robert Joseph Evans
>            Assignee: Boyang Jerry Peng
>         Attachments: Resource Aware Scheduler for Storm.pdf
>
>
> In a multi-tenant environment we would like to be able to give individual 
> users a guarantee of how much CPU/Memory/Network they will be able to use in 
> a cluster.  We would also like to know which topologies a user feels are the 
> most important to keep running if there are not enough resources to run all 
> of their topologies.
> Each user should be able to specify if their topology is production, staging, 
> or development. Within each of those categories a user should be able to give 
> a topology a priority, 0 to 10 with 10 being the highest priority (or 
> something like this).
> If there are not enough resources on a cluster to run a topology assume this 
> topology is running using resources and find the user that is most over their 
> guaranteed resources.  Shoot the lowest priority topology for that user, and 
> repeat until, this topology is able to run, or this topology would be the one 
> shot.   Ideally we don't actually shoot anything until we know that we would 
> have made enough room.
> If the cluster is over-subscribed and everyone is under their guarantee, and 
> this topology would not put the user over their guarantee.  Shoot the lowest 
> priority topology in this workers resource pool until there is enough room to 
> run the topology or this topology is the one that would be shot.  We might 
> also want to think about what to do if we are going to shoot a production 
> topology in an oversubscribed case, and perhaps we can shoot a non-production 
> topology instead even if the other user is not over their guarantee.



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

Reply via email to