Subramaniam Venkatraman Krishnan commented on YARN-2248:

Hi [~keyki], we have been working on adding support for move for sometime in 
Capacity Scheduler as part of YARN-2378 (originally YARN-1707) and [~vvasudev] 
was kind enough to point out that you were doing the same. To prevent 
duplication, I suggest we merge our work.

I looked at your patch & we are doing essentially the same thing (which was 
good validation for both of us :)). Based on [~leftnoteasy]'s [feedback | 
https://issues.apache.org/jira/browse/YARN-2378?focusedCommentId=14087723], I 
think it would be easiest if I merged your metrics test with the patch I have. 
Would that be OK?

> Capacity Scheduler changes for moving apps between queues
> ---------------------------------------------------------
>                 Key: YARN-2248
>                 URL: https://issues.apache.org/jira/browse/YARN-2248
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: capacityscheduler
>            Reporter: Janos Matyas
>            Assignee: Janos Matyas
>             Fix For: 2.6.0
>         Attachments: YARN-2248-1.patch, YARN-2248-2.patch, YARN-2248-3.patch
> We would like to have the capability (same as the Fair Scheduler has) to move 
> applications between queues. 
> We have made a baseline implementation and tests to start with - and we would 
> like the community to review, come up with suggestions and finally have this 
> contributed. 
> The current implementation is available for 2.4.1 - so the first thing is 
> that we'd need to identify the target version as there are differences 
> between 2.4.* and 3.* interfaces.
> The story behind is available at 
> http://blog.sequenceiq.com/blog/2014/07/02/move-applications-between-queues/ 
> and the baseline implementation and test at:
> https://github.com/sequenceiq/hadoop-common/blob/branch-2.4.1/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/a/ExtendedCapacityScheduler.java#L924
> https://github.com/sequenceiq/hadoop-common/blob/branch-2.4.1/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/a/TestExtendedCapacitySchedulerAppMove.java

This message was sent by Atlassian JIRA

Reply via email to