hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Akira AJISAKA (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-4543) TestNodeStatusUpdater.testStopReentrant fails + JUnit misusage
Date Fri, 29 Jan 2016 06:45:39 GMT

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

Akira AJISAKA commented on YARN-4543:
-------------------------------------

+1

> TestNodeStatusUpdater.testStopReentrant fails + JUnit misusage
> --------------------------------------------------------------
>
>                 Key: YARN-4543
>                 URL: https://issues.apache.org/jira/browse/YARN-4543
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager
>            Reporter: Akihiro Suda
>            Assignee: Akihiro Suda
>            Priority: Minor
>         Attachments: 0001-YARN-4543.patch, YARN-4543-1.patch
>
>
> {panel}
> TestNodeStatusUpdater.testStopReentrant:1269 expected:<0> but was:<1>
> {panel}
> https://github.com/apache/hadoop/blob/4ac6799d4a8b071e0d367c2d709e84d8ea06942d/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestNodeStatusUpdater.java#L1269
> The corresponding  JUnit assertion code is: 
> {panel}
>  Assert.assertEquals(numCleanups.get(), 1);
> {panel}
> It seems that the 1st arg and the 2nd one should be swapped.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message