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

Bikas Saha commented on YARN-1506:
----------------------------------

Does a node update forced by the admin need to be persisted across RM 
restarts/failover?

UpdateNodeResourceWhenUnusableTransition should also update the resource IMO so 
that its got the correct value when the node becomes usable again. Also, It may 
be useful to take the common pieces of code between update when 
running/not-running to be in a shared place. E.g. the update of the node 
resource member var.

Do we need to remember in the RMNode that the node resource was forced to a 
certain value. E.g. when a bad node re-registers with the RM then it may send 
its real resource which could override the admin value.

Since the capacity scheduler is the default scheduler that runs in every test, 
it would be good to have a test for it.

> Replace set resource change on RMNode/SchedulerNode directly with event 
> notification.
> -------------------------------------------------------------------------------------
>
>                 Key: YARN-1506
>                 URL: https://issues.apache.org/jira/browse/YARN-1506
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager, scheduler
>            Reporter: Junping Du
>            Assignee: Junping Du
>         Attachments: YARN-1506-v1.patch, YARN-1506-v2.patch, 
> YARN-1506-v3.patch, YARN-1506-v4.patch, YARN-1506-v5.patch, 
> YARN-1506-v6.patch, YARN-1506-v7.patch, YARN-1506-v8.patch, YARN-1506-v9.patch
>
>
> According to Vinod's comments on YARN-312 
> (https://issues.apache.org/jira/browse/YARN-312?focusedCommentId=13846087&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13846087),
>  we should replace RMNode.setResourceOption() with some resource change event.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to