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

Chuan Liu commented on YARN-1077:
---------------------------------

In the new patch, I removed the following code because writer is writing to 
shellFile but later the file outputsteam opens the file without append mode. So 
the code is simply dead code.

{code:java}
       shellFile = Shell.appendScriptExtension(tmpDir, "hello");
-      String timeoutCommand = Shell.WINDOWS ? "@echo \"hello\"" :
-        "echo \"hello\"";
-      PrintWriter writer = new PrintWriter(new FileOutputStream(shellFile));
-      FileUtil.setExecutable(shellFile, true);
-      writer.println(timeoutCommand);
-      writer.close();
       Map<Path, List<String>> resources =
           new HashMap<Path, List<String>>();
       FileOutputStream fos = new FileOutputStream(shellFile);
{code}
                
> TestContainerLaunch fails on Windows
> ------------------------------------
>
>                 Key: YARN-1077
>                 URL: https://issues.apache.org/jira/browse/YARN-1077
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 3.0.0, 2.3.0
>            Reporter: Chuan Liu
>            Assignee: Chuan Liu
>            Priority: Minor
>         Attachments: YARN-1077.2.patch, YARN-1077.3.patch, YARN-1077.patch
>
>
> Several cases in this unit tests fail on Windows. (Append error log at the 
> end.)
> testInvalidEnvSyntaxDiagnostics fails because the difference between cmd and 
> bash script error handling. If some command fails in the cmd script, cmd will 
> continue execute the the rest of the script command. Error handling needs to 
> be explicitly carried out in the script file. The error code of the last 
> command will be returned as the error code of the whole script. In this test, 
> some error happened in the middle of the cmd script, the test expect an 
> exception and non-zero error code. In the cmd script, the intermediate errors 
> are ignored. The last command "call" succeeded and there is no exception.
> testContainerLaunchStdoutAndStderrDiagnostics fails due to wrong cmd commands 
> used by the test.
> testContainerEnvVariables and testDelayedKill fail due to a regression from 
> YARN-906.
> {noformat}
> -------------------------------------------------------------------------------
> Test set: 
> org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch
> -------------------------------------------------------------------------------
> Tests run: 7, Failures: 4, Errors: 0, Skipped: 0, Time elapsed: 11.526 sec 
> <<< FAILURE!
> testInvalidEnvSyntaxDiagnostics(org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch)
>   Time elapsed: 583 sec  <<< FAILURE!
> junit.framework.AssertionFailedError: Should catch exception
>       at junit.framework.Assert.fail(Assert.java:50)
>       at 
> org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch.testInvalidEnvSyntaxDiagnostics(TestContainerLaunch.java:269)
> ...
> testContainerLaunchStdoutAndStderrDiagnostics(org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch)
>   Time elapsed: 561 sec  <<< FAILURE!
> junit.framework.AssertionFailedError: Should catch exception
>       at junit.framework.Assert.fail(Assert.java:50)
>       at 
> org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch.testContainerLaunchStdoutAndStderrDiagnostics(TestContainerLaunch.java:314)
> ...
> testContainerEnvVariables(org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch)
>   Time elapsed: 4136 sec  <<< FAILURE!
> junit.framework.AssertionFailedError: expected:<137> but was:<143>
>       at junit.framework.Assert.fail(Assert.java:50)
>       at junit.framework.Assert.failNotEquals(Assert.java:287)
>       at junit.framework.Assert.assertEquals(Assert.java:67)
>       at junit.framework.Assert.assertEquals(Assert.java:199)
>       at junit.framework.Assert.assertEquals(Assert.java:205)
>       at 
> org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch.testContainerEnvVariables(TestContainerLaunch.java:500)
> ...
> testDelayedKill(org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch)
>   Time elapsed: 2744 sec  <<< FAILURE!
> junit.framework.AssertionFailedError: expected:<137> but was:<143>
>       at junit.framework.Assert.fail(Assert.java:50)
>       at junit.framework.Assert.failNotEquals(Assert.java:287)
>       at junit.framework.Assert.assertEquals(Assert.java:67)
>       at junit.framework.Assert.assertEquals(Assert.java:199)
>       at junit.framework.Assert.assertEquals(Assert.java:205)
>       at 
> org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher.TestContainerLaunch.testDelayedKill(TestContainerLaunch.java:601)
> ...
> {noformat}

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