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

Arun Suresh commented on YARN-4511:
-----------------------------------

So with regard to the extra synchronization in {{swapContainer()}}, please make 
sure to test it either via SLS / running actual jobs - since there can be a 
possibility of deadlock (ok - not dead lock, but quite perceptible slow down) 
if the node in question tries to perform a heartbeat during the allocate call.
I am thinking maybe we split this patch:
* Keep this patch to track modifications required for addition of the 
OverAllocationInfo
* Have another patch for the changes to the {{swapContainer}} etc. That way, we 
can actually see how the modifications to the ContainerUpdateContext are being 
used.
Thoughts?

> Common scheduler changes supporting scheduler-specific implementations
> ----------------------------------------------------------------------
>
>                 Key: YARN-4511
>                 URL: https://issues.apache.org/jira/browse/YARN-4511
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Wangda Tan
>            Assignee: Haibo Chen
>         Attachments: YARN-4511-YARN-1011.00.patch, 
> YARN-4511-YARN-1011.01.patch, YARN-4511-YARN-1011.02.patch, 
> YARN-4511-YARN-1011.03.patch, YARN-4511-YARN-1011.04.patch, 
> YARN-4511-YARN-1011.05.patch, YARN-4511-YARN-1011.06.patch, 
> YARN-4511-YARN-1011.07.patch, YARN-4511-YARN-1011.08.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to