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

Haibo Chen commented on YARN-8388:
----------------------------------

Two minor comments

1) Let's add a comment above ` when(cgroups.getPathForCGroup(any(), 
any())).thenReturn("10000");` that 10000 will be passed to the sleep command. 
This is a little hacky, so worth a comment I think.

2) there is a misspelling, `aviod` -> 'avoid'

> TestCGroupElasticMemoryController.testNormalExit() hangs on Linux
> -----------------------------------------------------------------
>
>                 Key: YARN-8388
>                 URL: https://issues.apache.org/jira/browse/YARN-8388
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>    Affects Versions: 3.2.0
>            Reporter: Haibo Chen
>            Assignee: Miklos Szegedi
>            Priority: Major
>         Attachments: YARN-8388.000.patch
>
>
> YARN-8375 disables the unit test on Linux. But given that we will be running 
> the CGroupElasticMemoryController on Linux, we need to figure out why it is 
> hanging and ideally fix it.
>  
>  



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