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

Konstantinos Karanasos commented on YARN-5171:
----------------------------------------------

Naming in the latest patch (v011) looks reasonable to me.

Please don't forget to add some comments explaining what "allocated remotely" 
means (i.e., containers allocated by a scheduler other than YARN's RM, such as 
the distributed schedulers of YARN-2877).
Also, let's create a JIRA, as [~subru] suggests, for refactoring the code so 
that we always use the new (add/remove)RMContainer methods, and link it to this 
one.

> Extend DistributedSchedulerProtocol to notify RM of containers allocated by 
> the Node
> ------------------------------------------------------------------------------------
>
>                 Key: YARN-5171
>                 URL: https://issues.apache.org/jira/browse/YARN-5171
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Arun Suresh
>            Assignee: Inigo Goiri
>         Attachments: YARN-5171.000.patch, YARN-5171.001.patch, 
> YARN-5171.002.patch, YARN-5171.003.patch, YARN-5171.004.patch, 
> YARN-5171.005.patch, YARN-5171.006.patch, YARN-5171.007.patch, 
> YARN-5171.008.patch, YARN-5171.009.patch, YARN-5171.010.patch, 
> YARN-5171.011.patch
>
>
> Currently, the RM does not know about Containers allocated by the 
> OpportunisticContainerAllocator on the NM. This JIRA proposes to extend the 
> Distributed Scheduler request interceptor and the protocol to notify the RM 
> of new containers as and when they are allocated at the NM. The 
> {{RMContainer}} should also be extended to expose the {{ExecutionType}} of 
> the container.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to