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 om...@apache.org on 2011/03/04 05:21:12 UTC

svn commit: r1077500 - /hadoop/common/branches/branch-0.20-security-patches/src/test/org/apache/hadoop/hdfs/server/datanode/TestDiskError.java

Author: omalley
Date: Fri Mar  4 04:21:11 2011
New Revision: 1077500

URL: http://svn.apache.org/viewvc?rev=1077500&view=rev
Log:
commit 6ad824b2592f9bf2f5076141d3620c0382c0bcd4
Author: Luke Lu <ll...@yahoo-inc.com>
Date:   Fri Jun 11 09:38:11 2010 -0700

    HDFS:1101 https://issues.apache.org/jira/secure/attachment/12446878/hdfs-1101-y20s-v1-d1.patch

Modified:
    hadoop/common/branches/branch-0.20-security-patches/src/test/org/apache/hadoop/hdfs/server/datanode/TestDiskError.java

Modified: hadoop/common/branches/branch-0.20-security-patches/src/test/org/apache/hadoop/hdfs/server/datanode/TestDiskError.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-patches/src/test/org/apache/hadoop/hdfs/server/datanode/TestDiskError.java?rev=1077500&r1=1077499&r2=1077500&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.20-security-patches/src/test/org/apache/hadoop/hdfs/server/datanode/TestDiskError.java (original)
+++ hadoop/common/branches/branch-0.20-security-patches/src/test/org/apache/hadoop/hdfs/server/datanode/TestDiskError.java Fri Mar  4 04:21:11 2011
@@ -167,13 +167,13 @@ public class TestDiskError extends TestC
       
       // Check permissions on directories in 'dfs.data.dir'
       FileSystem localFS = FileSystem.getLocal(conf);
-      String[] dataDirs = conf.getStrings(DataNode.DATA_DIR_KEY);
-      for (String dir : dataDirs) {
-        Path dataDir = new Path(dir);
-        FsPermission actual = localFS.getFileStatus(dataDir).getPermission(); 
-        assertEquals("Permission for dir: " + dataDir + ", is " + actual + 
-                         ", while expected is " + expected, 
-                     expected, actual);
+      for (DataNode dn : cluster.getDataNodes()) {
+        String[] dataDirs = dn.getConf().getStrings(DataNode.DATA_DIR_KEY);
+        for (String dir : dataDirs) {
+          Path dataDir = new Path(dir);
+          FsPermission actual = localFS.getFileStatus(dataDir).getPermission();
+          assertEquals("Permission for dir: " + dataDir, expected, actual);
+        }
       }
     } finally {
       if (cluster != null)