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

Hudson commented on YARN-10277:
-------------------------------

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #18381 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/18381/])
YARN-10277. CapacityScheduler test TestUserGroupMappingPlacementRule (snemeth: 
rev 33d3df7ee2b3942560e9affe1409854ac3e8bb96)
* (edit) 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/placement/TestUserGroupMappingPlacementRule.java


> CapacityScheduler test TestUserGroupMappingPlacementRule should build proper 
> hierarchy
> --------------------------------------------------------------------------------------
>
>                 Key: YARN-10277
>                 URL: https://issues.apache.org/jira/browse/YARN-10277
>             Project: Hadoop YARN
>          Issue Type: Task
>            Reporter: Gergely Pollak
>            Assignee: Szilard Nemeth
>            Priority: Major
>             Fix For: 3.4.0
>
>         Attachments: YARN-10277.001.patch, YARN-10277.002.patch, 
> YARN-10277.003.patch, YARN-10277.branch-3.3.001.patch
>
>
> Since the CapacityScheduler internal implementation depends more and more on 
> queue being hierarchical, the test gets really hard to maintain. A lot of 
> test cases were failing because they used non existing queues, but the older 
> placement rule solution ignored missing parents, but since the leaf queue 
> change in CS, we must be able to get a full path for any queue, since all 
> queues are referenced by their full path.
> This test should reflect this and instead of creating and expecting the 
> existance of fictional queues, it should create a proper queue hierarchy, 
> with a way to describe it better. 
> Currently we set up a bunch of mockito "when" statements to simulate the 
> queue behavior, but this is a hassle to maintain, and easy to miss a few 
> method.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
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