You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by su...@apache.org on 2012/11/04 20:11:33 UTC

svn commit: r1405615 - in /hadoop/common/branches/branch-1-win: CHANGES.branch-1-win.txt src/test/org/apache/hadoop/mapred/TestJobStatusPersistency.java

Author: suresh
Date: Sun Nov  4 19:11:33 2012
New Revision: 1405615

URL: http://svn.apache.org/viewvc?rev=1405615&view=rev
Log:
HADOOP-9007. TestJobStatusPersistency.testJobStoreDisablingWithInvalidPath failure. Contributed by Chuan Liu.

Modified:
    hadoop/common/branches/branch-1-win/CHANGES.branch-1-win.txt
    hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobStatusPersistency.java

Modified: hadoop/common/branches/branch-1-win/CHANGES.branch-1-win.txt
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/CHANGES.branch-1-win.txt?rev=1405615&r1=1405614&r2=1405615&view=diff
==============================================================================
--- hadoop/common/branches/branch-1-win/CHANGES.branch-1-win.txt (original)
+++ hadoop/common/branches/branch-1-win/CHANGES.branch-1-win.txt Sun Nov  4 19:11:33 2012
@@ -205,3 +205,6 @@ Branch-hadoop-1-win - unreleased
 
     HADOOP-8983. Minor fixes to windows batch scripts to accept alternate 
     config directory location. (Ramya Bharathi Nimmagadda via suresh)
+
+    HADOOP-9007. TestJobStatusPersistency.testJobStoreDisablingWithInvalidPath
+    failure. (Chuan Liu via suresh)

Modified: hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobStatusPersistency.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobStatusPersistency.java?rev=1405615&r1=1405614&r2=1405615&view=diff
==============================================================================
--- hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobStatusPersistency.java (original)
+++ hadoop/common/branches/branch-1-win/src/test/org/apache/hadoop/mapred/TestJobStatusPersistency.java Sun Nov  4 19:11:33 2012
@@ -28,6 +28,7 @@ import org.apache.hadoop.fs.FileSystem;
 import org.apache.hadoop.fs.Path;
 import org.apache.hadoop.io.LongWritable;
 import org.apache.hadoop.io.Text;
+import org.apache.hadoop.fs.FileUtil;
 
 public class TestJobStatusPersistency extends ClusterMapReduceTestCase {
   static final Path TEST_DIR = 
@@ -157,7 +158,7 @@ public class TestJobStatusPersistency ex
       }
 
       if (fs.mkdirs(new Path(TEST_DIR, parent))) {
-        if (!(new File(parent.toUri().getPath()).setWritable(false, false))) {
+        if (FileUtil.chmod(parent.toUri().getPath(), "-w") != 0) {
           fail("Cannot chmod parent!");
         }
       } else {
@@ -176,7 +177,7 @@ public class TestJobStatusPersistency ex
         started = false;
       }
     } finally {
-      new File(parent.toUri().getPath()).setWritable(true, false);
+      FileUtil.chmod(parent.toUri().getPath(), "+w");
     }
   }
 }