[ 
https://issues.apache.org/jira/browse/YARN-909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chris Nauroth updated YARN-909:
-------------------------------

    Attachment: YARN-909.3.patch

Actually, I was thinking we could put the {{assumeTrue}} call inside the 
{{Before}} method.  This way, it's a very small patch.  It also helps ease 
maintenance, because when people add new tests to this suite, they won't need 
to remember to call {{assumeTrue}}.

I apologize if I'm not communicating this clearly.  I'm attaching a patch that 
shows it.  Sometimes code is easier.  :-)

Does this look good to you?
                
> Disable TestLinuxContainerExecutorWithMocks on Windows
> ------------------------------------------------------
>
>                 Key: YARN-909
>                 URL: https://issues.apache.org/jira/browse/YARN-909
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Chuan Liu
>            Assignee: Chuan Liu
>            Priority: Minor
>         Attachments: YARN-909.2.patch, YARN-909.3.patch, YARN-909.patch
>
>
> This unit test tests a Linux specific feature. We should skip this unit test 
> on Windows. A similar unit test 'TestLinuxContainerExecutor' was already 
> skipped when running on Windows.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to