Szilard Nemeth commented on YARN-10330:

Thanks [~pbacsko],
Latest patch LGTM, committed to trunk.
As discussed, please upload a patch for branch-3.3 as well.

> Add missing test scenarios to TestUserGroupMappingPlacementRule and 
> TestAppNameMappingPlacementRule
> ---------------------------------------------------------------------------------------------------
>                 Key: YARN-10330
>                 URL: https://issues.apache.org/jira/browse/YARN-10330
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: capacity scheduler, capacityscheduler, test
>            Reporter: Peter Bacsko
>            Assignee: Peter Bacsko
>            Priority: Major
>             Fix For: 3.4.0
>         Attachments: YARN-10330-001.patch, YARN-10330-002.patch, 
> YARN-10330-003.patch, YARN-10330-004.patch
> After running {{TestUserGroupMappingPlacementRule}} with EclEmma, it turned 
> out that there are at least 8-10 missing test scenarios that are not covered. 
> Since we're planning to enhance mapping rule logic with extra features, it is 
> crucial to have good coverage so that we can verify backward compatibility.

This message was sent by Atlassian Jira

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