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 ji...@apache.org on 2008/03/21 22:39:58 UTC

svn commit: r639870 - in /hadoop/core/trunk: CHANGES.txt src/java/org/apache/hadoop/dfs/FSNamesystem.java

Author: jimk
Date: Fri Mar 21 14:39:56 2008
New Revision: 639870

URL: http://svn.apache.org/viewvc?rev=639870&view=rev
Log:
HADOOP-3066. Should not require superuser privilege to query if hdfs is in safe mode

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=639870&r1=639869&r2=639870&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Fri Mar 21 14:39:56 2008
@@ -326,6 +326,9 @@
 
     HADOOP-3012. dfs -mv file to user home directory throws exception if 
     the user home directory does not exist. (Mahadev Konar via dhruba)
+    
+    HADOOP-3066. Should not require superuser privilege to query if hdfs is in
+    safe mode (jimk)
 
 Release 0.16.2 - Unreleased
 

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java?rev=639870&r1=639869&r2=639870&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/dfs/FSNamesystem.java Fri Mar 21 14:39:56 2008
@@ -3936,15 +3936,16 @@
   }
     
   boolean setSafeMode(SafeModeAction action) throws IOException {
-    checkSuperuserPrivilege();
-    switch(action) {
-    case SAFEMODE_LEAVE: // leave safe mode
-      leaveSafeMode(false);
-      break;
-    case SAFEMODE_ENTER: // enter safe mode
-      enterSafeMode();
-      break;
-    case SAFEMODE_GET: // get safe mode
+    if (action != SafeModeAction.SAFEMODE_GET) {
+      checkSuperuserPrivilege();
+      switch(action) {
+      case SAFEMODE_LEAVE: // leave safe mode
+        leaveSafeMode(false);
+        break;
+      case SAFEMODE_ENTER: // enter safe mode
+        enterSafeMode();
+        break;
+      }
     }
     return isInSafeMode();
   }