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 cu...@apache.org on 2006/10/31 23:17:47 UTC

svn commit: r469678 - in /lucene/hadoop/trunk: CHANGES.txt src/java/org/apache/hadoop/dfs/DFSck.java

Author: cutting
Date: Tue Oct 31 14:17:46 2006
New Revision: 469678

URL: http://svn.apache.org/viewvc?view=rev&rev=469678
Log:
HADOOP-651.  Fix DFSCk to correctly pass parameters to the servlet on the namenode.  Contributed by Milind.

Modified:
    lucene/hadoop/trunk/CHANGES.txt
    lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DFSck.java

Modified: lucene/hadoop/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/CHANGES.txt?view=diff&rev=469678&r1=469677&r2=469678
==============================================================================
--- lucene/hadoop/trunk/CHANGES.txt (original)
+++ lucene/hadoop/trunk/CHANGES.txt Tue Oct 31 14:17:46 2006
@@ -97,6 +97,9 @@
 26. HADOOP-90.  Permit dfs.name.dir to list multiple directories,
     where namenode data is to be replicated. (Milind Bhandarkar via cutting)
 
+27. HADOOP-651.  Fix DFSCk to correctly pass parameters to the servlet
+    on the namenode.  (Milind Bhandarkar via cutting)
+
 
 Release 0.7.2 - 2006-10-18
 

Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DFSck.java
URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DFSck.java?view=diff&rev=469678&r1=469677&r2=469678
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DFSck.java (original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/dfs/DFSck.java Tue Oct 31 14:17:46 2006
@@ -99,12 +99,12 @@
       if (!args[idx].startsWith("-")) { dir = args[idx]; break; }
     }
     url.append(URLEncoder.encode(dir, "UTF-8"));
-    for (int idx = 1; idx < args.length; idx++) {
-      if (args[idx].equals("-move")) { url.append("&move"); }
-      if (args[idx].equals("-delete")) { url.append("&delete"); }
-      if (args[idx].equals("-files")) { url.append("&files"); }
-      if (args[idx].equals("-blocks")) { url.append("&blocks"); }
-      if (args[idx].equals("-locations")) { url.append("&locations"); }
+    for (int idx = 0; idx < args.length; idx++) {
+      if (args[idx].equals("-move")) { url.append("&move=1"); }
+      else if (args[idx].equals("-delete")) { url.append("&delete=1"); }
+      else if (args[idx].equals("-files")) { url.append("&files=1"); }
+      else if (args[idx].equals("-blocks")) { url.append("&blocks=1"); }
+      else if (args[idx].equals("-locations")) { url.append("&locations=1"); }
     }
     URL path = new URL(url.toString());
     URLConnection connection = path.openConnection();