You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-commits@hadoop.apache.org by el...@apache.org on 2011/08/15 23:47:00 UTC
svn commit: r1158025 - in /hadoop/common/trunk/hdfs: CHANGES.txt
src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java
Author: eli
Date: Mon Aug 15 21:47:00 2011
New Revision: 1158025
URL: http://svn.apache.org/viewvc?rev=1158025&view=rev
Log:
HDFS-2233. Add WebUI tests with URI reserved chars. Contributed by Eli Collins
Modified:
hadoop/common/trunk/hdfs/CHANGES.txt
hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java
Modified: hadoop/common/trunk/hdfs/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hdfs/CHANGES.txt?rev=1158025&r1=1158024&r2=1158025&view=diff
==============================================================================
--- hadoop/common/trunk/hdfs/CHANGES.txt (original)
+++ hadoop/common/trunk/hdfs/CHANGES.txt Mon Aug 15 21:47:00 2011
@@ -663,6 +663,8 @@ Trunk (unreleased changes)
HDFS-2237. Change UnderReplicatedBlocks from public to package private.
(szetszwo)
+ HDFS-2233. Add WebUI tests with URI reserved chars. (eli)
+
OPTIMIZATIONS
HDFS-1458. Improve checkpoint performance by avoiding unnecessary image
Modified: hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java?rev=1158025&r1=1158024&r2=1158025&view=diff
==============================================================================
--- hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java (original)
+++ hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java Mon Aug 15 21:47:00 2011
@@ -82,31 +82,22 @@ public class TestDatanodeJsp {
try {
cluster = new MiniDFSCluster.Builder(CONF).build();
cluster.waitActive();
-
- testViewingFile(cluster, "/test-file", false);
- testViewingFile(cluster, "/tmp/test-file", false);
- testViewingFile(cluster, "/tmp/test-file%with goofy&characters", false);
-
- testViewingFile(cluster, "/test-file", true);
- testViewingFile(cluster, "/tmp/test-file", true);
- testViewingFile(cluster, "/tmp/test-file%with goofy&characters", true);
-
- testViewingFile(cluster, "/foo bar", true);
- testViewingFile(cluster, "/foo+bar", true);
- testViewingFile(cluster, "/foo;bar", true);
- testViewingFile(cluster, "/foo=bar", true);
- testViewingFile(cluster, "/foo,bar", true);
- testViewingFile(cluster, "/foo?bar", true);
- testViewingFile(cluster, "/foo\">bar", true);
-
- testViewingFile(cluster, "/foo bar", false);
- // See HDFS-2233
- //testViewingFile(cluster, "/foo+bar", false);
- //testViewingFile(cluster, "/foo;bar", false);
- testViewingFile(cluster, "/foo=bar", false);
- testViewingFile(cluster, "/foo,bar", false);
- testViewingFile(cluster, "/foo?bar", false);
- testViewingFile(cluster, "/foo\">bar", false);
+ String paths[] = {
+ "/test-file",
+ "/tmp/test-file",
+ "/tmp/test-file%with goofy&characters",
+ "/foo bar/foo bar",
+ "/foo+bar/foo+bar",
+ "/foo;bar/foo;bar",
+ "/foo=bar/foo=bar",
+ "/foo,bar/foo,bar",
+ "/foo?bar/foo?bar",
+ "/foo\">bar/foo\">bar"
+ };
+ for (String p : paths) {
+ testViewingFile(cluster, p, false);
+ testViewingFile(cluster, p, true);
+ }
} finally {
if (cluster != null) {
cluster.shutdown();