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

Eric Payne commented on YARN-8421:
----------------------------------

[~kyungwan nam], Thank you for providing the fix for this problem. The fix 
looks good and the unit test is doing a good job of testing what I would expect 
it to test. The failed unit tests in the latest pre-commit build 
({{TestAMRestart}} / {{TestQueueManagementDynamicEditPolicy}}) are not failing 
for me in my local build environment.

The only minor problem with the latest patch is that the parameters to the 
assertions in the test are backwards. That is, the "expected" value should come 
first and the "actual" value should come second.

> when moving app, activeUsers is increased, even though app does not have 
> outstanding request 
> ---------------------------------------------------------------------------------------------
>
>                 Key: YARN-8421
>                 URL: https://issues.apache.org/jira/browse/YARN-8421
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.8.4
>            Reporter: kyungwan nam
>            Priority: Major
>         Attachments: YARN-8421.001.patch, YARN-8421.002.patch
>
>
> all containers for app1 have been allocated.
> move app1 from default Queue to test Queue as follows.
> {code}
>   yarn rmadmin application -movetoqueue app1 -queue test
> {code}
> _activeUsers_ of the test Queue is increased even though app1 which does not 
> have outstanding request.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to