diff --git a/hadoop-mapreduce-project/CHANGES.txt b/hadoop-mapreduce-project/CHANGES.txt index ea8d2a95399..bfd656c3886 100644 --- a/hadoop-mapreduce-project/CHANGES.txt +++ b/hadoop-mapreduce-project/CHANGES.txt @@ -484,6 +484,9 @@ Release 2.1.0-beta - 2013-07-02 MAPREDUCE-5360. TestMRJobClient fails on Windows due to path format. (Chuan Liu via cnauroth) + MAPREDUCE-5366. TestMRAsyncDiskService fails on Windows. (Chuan Liu via + cnauroth) + MAPREDUCE-5291. Change MR App to use updated property names in container-log4j.properties. (Zhijie Shen via sseth) diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapreduce/util/TestMRAsyncDiskService.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapreduce/util/TestMRAsyncDiskService.java index 3e05962d865..e1849a3ce9c 100644 --- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapreduce/util/TestMRAsyncDiskService.java +++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapreduce/util/TestMRAsyncDiskService.java @@ -59,8 +59,8 @@ public class TestMRAsyncDiskService extends TestCase { pathname = (new Path(pathname)).toUri().getPath(); cwd = (new Path(cwd)).toUri().getPath(); - String [] cwdParts = cwd.split(File.separator); - String [] pathParts = pathname.split(File.separator); + String [] cwdParts = cwd.split(Path.SEPARATOR); + String [] pathParts = pathname.split(Path.SEPARATOR); // There are three possible cases: // 1) pathname and cwd are equal. Return '.' @@ -94,18 +94,18 @@ public class TestMRAsyncDiskService extends TestCase { int parentDirsRequired = cwdParts.length - common; for (int i = 0; i < parentDirsRequired; i++) { sb.append(".."); - sb.append(File.separator); + sb.append(Path.SEPARATOR); } // Then append all non-common parts of 'pathname' itself. for (int i = common; i < pathParts.length; i++) { sb.append(pathParts[i]); - sb.append(File.separator); + sb.append(Path.SEPARATOR); } // Don't end with a '/'. String s = sb.toString(); - if (s.endsWith(File.separator)) { + if (s.endsWith(Path.SEPARATOR)) { s = s.substring(0, s.length() - 1); }