YARN-2711. Fixed TestDefaultContainerExecutor#testContainerLaunchError failure on Windows. Contributed by Varun Vasudev.
This commit is contained in:
parent
80bb7d4794
commit
1cd088fd9d
@ -829,6 +829,9 @@ Release 2.6.0 - UNRELEASED
|
||||
YARN-2707. Potential null dereference in FSDownload (Gera Shegalov via
|
||||
jlowe)
|
||||
|
||||
YARN-2711. Fixed TestDefaultContainerExecutor#testContainerLaunchError failure on
|
||||
Windows. (Varun Vasudev via zjshen)
|
||||
|
||||
Release 2.5.1 - 2014-09-05
|
||||
|
||||
INCOMPATIBLE CHANGES
|
||||
|
@ -27,10 +27,12 @@
|
||||
import static org.mockito.Mockito.doAnswer;
|
||||
import static org.junit.Assert.assertTrue;
|
||||
|
||||
import java.io.BufferedWriter;
|
||||
import java.io.DataOutputStream;
|
||||
import java.io.File;
|
||||
import java.io.FileNotFoundException;
|
||||
import java.io.FileReader;
|
||||
import java.io.FileWriter;
|
||||
import java.io.InputStream;
|
||||
import java.io.IOException;
|
||||
import java.io.LineNumberReader;
|
||||
@ -59,6 +61,7 @@
|
||||
import org.apache.hadoop.io.DataOutputBuffer;
|
||||
import org.apache.hadoop.security.Credentials;
|
||||
import org.apache.hadoop.util.Progressable;
|
||||
import org.apache.hadoop.util.Shell;
|
||||
import org.apache.hadoop.yarn.api.records.ApplicationAttemptId;
|
||||
import org.apache.hadoop.yarn.api.records.ApplicationId;
|
||||
import org.apache.hadoop.yarn.api.records.ContainerId;
|
||||
@ -118,7 +121,7 @@ DataOutputBuffer mockStream(
|
||||
}
|
||||
*/
|
||||
|
||||
private static final Path BASE_TMP_PATH = new Path("target",
|
||||
private static Path BASE_TMP_PATH = new Path("target",
|
||||
TestDefaultContainerExecutor.class.getSimpleName());
|
||||
|
||||
@AfterClass
|
||||
@ -217,6 +220,12 @@ public void testDirPermissions() throws Exception {
|
||||
public void testContainerLaunchError()
|
||||
throws IOException, InterruptedException {
|
||||
|
||||
if (Shell.WINDOWS) {
|
||||
BASE_TMP_PATH =
|
||||
new Path(new File("target").getAbsolutePath(),
|
||||
TestDefaultContainerExecutor.class.getSimpleName());
|
||||
}
|
||||
|
||||
Path localDir = new Path(BASE_TMP_PATH, "localDir");
|
||||
List<String> localDirs = new ArrayList<String>();
|
||||
localDirs.add(localDir.toString());
|
||||
@ -284,6 +293,17 @@ public Object answer(InvocationOnMock invocationOnMock)
|
||||
|
||||
Path scriptPath = new Path("file:///bin/echo");
|
||||
Path tokensPath = new Path("file:///dev/null");
|
||||
if (Shell.WINDOWS) {
|
||||
File tmp = new File(BASE_TMP_PATH.toString(), "test_echo.cmd");
|
||||
BufferedWriter output = new BufferedWriter(new FileWriter(tmp));
|
||||
output.write("Exit 1");
|
||||
output.write("Echo No such file or directory 1>&2");
|
||||
output.close();
|
||||
scriptPath = new Path(tmp.getAbsolutePath());
|
||||
tmp = new File(BASE_TMP_PATH.toString(), "tokens");
|
||||
tmp.createNewFile();
|
||||
tokensPath = new Path(tmp.getAbsolutePath());
|
||||
}
|
||||
Path workDir = localDir;
|
||||
Path pidFile = new Path(workDir, "pid.txt");
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user