[ https://issues.apache.org/jira/browse/YARN-2143?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14042251#comment-14042251 ]
Wangda Tan commented on YARN-2143: ---------------------------------- Hi Sunil, Thanks for your comment, I took a look at FairScheduler/CapacityScheduler again, I think completedContainer looks more like what we need put in AbstractYarnScheduler. Both of warnOrKillContainer and killContainer invokes completedContainer. This issue is created because of YARN-1408, I feel we'd better leave this alone here and wait for some feedbacks from community, we can add ResourceRequest logic separately in Fair/CapacityScheduler for now, and refactor it in the future. Thanks, > Merge common killContainer logic of Fair/Capacity scheduler into > AbstractYarnScheduler > -------------------------------------------------------------------------------------- > > Key: YARN-2143 > URL: https://issues.apache.org/jira/browse/YARN-2143 > Project: Hadoop YARN > Issue Type: Task > Components: resourcemanager, scheduler > Reporter: Wangda Tan > > Currently, CapacityScheduler has killContainer API inherited from > PreemptableResourceScheduler, and FairScheduler uses warnOrKillContainer to > do container preemption. We'd better to merge common code to kill container > into AbstractYarnScheduler. -- This message was sent by Atlassian JIRA (v6.2#6252)