tomscut commented on a change in pull request #4067:
URL: https://github.com/apache/hadoop/pull/4067#discussion_r826018545
##########
File path:
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestWebHDFS.java
##########
@@ -522,6 +523,38 @@ public void testCreateWithNoDN() throws Exception {
}
}
+ @Test
+ public void testWebHdfsCreateWithInvalidPath() throws Exception {
+ final Configuration conf = WebHdfsTestUtil.createConf();
+ cluster = new MiniDFSCluster.Builder(conf).numDataNodes(1).build();
Review comment:
> do you need a datanode, if you aren't writing any data, you can
survive with 0 datanodes as well? btw if not, the default is 1 only, you don't
need to explicitly put one there. IIRC
Thanks you @ayushtkn very much for the review. I fixed it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
