Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HDFS-10684. WebHDFS DataNode calls fail without parameter createparent. Contributed by John Zhuge. 

(cherry picked from commit fbdbbd57cd)
This commit is contained in:
Andrew Wang 2016-12-13 18:01:31 -08:00
parent 0af67aa200
commit 6b795c34d0
3 changed files with 42 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/CreateParentParam.java
View File

@ -39,7 +39,7 @@ public CreateParentParam(final Boolean value) {
* @param str a string representation of the parameter value.
*/
public CreateParentParam(final String str) {
this(DOMAIN.parse(str));
this(DOMAIN.parse(str == null ? DEFAULT : str));
}
@Override

2
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/web/resources/OverwriteParam.java
View File

@ -39,7 +39,7 @@ public OverwriteParam(final Boolean value) {
* @param str a string representation of the parameter value.
*/
public OverwriteParam(final String str) {
this(DOMAIN.parse(str));
super(DOMAIN, DOMAIN.parse(str == null ? DEFAULT : str));
}
@Override

40
hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestWebHdfsFileSystemContract.java
View File

@ -25,6 +25,7 @@
import java.io.InputStreamReader;
import java.net.HttpURLConnection;
import java.net.URL;
import java.text.MessageFormat;
import java.util.Arrays;
import java.util.Map;
import java.util.Random;
@ -49,6 +50,7 @@
import org.apache.hadoop.io.IOUtils;
import org.apache.hadoop.security.AccessControlException;
import org.apache.hadoop.security.UserGroupInformation;
import org.apache.hadoop.test.GenericTestUtils;
import org.junit.Assert;
import org.junit.Test;
@ -533,6 +535,44 @@ public void testResponseCode() throws IOException {
}
}
public void testDatanodeCreateMissingParameter() throws IOException {
final WebHdfsFileSystem webhdfs = (WebHdfsFileSystem) fs;
final Path testDir = new Path(MessageFormat.format("/test/{0}/{1}",
TestWebHdfsFileSystemContract.class,
GenericTestUtils.getMethodName()));
assertTrue(webhdfs.mkdirs(testDir));
for (String dnCreateParam : new String[]{
CreateFlagParam.NAME,
CreateParentParam.NAME,
OverwriteParam.NAME
}) {
final HttpOpParam.Op op = PutOpParam.Op.CREATE;
final Path newfile = new Path(testDir, "newfile_" + dnCreateParam);
final URL url = webhdfs.toUrl(op, newfile);
HttpURLConnection conn = (HttpURLConnection) url.openConnection();
conn.setRequestMethod(op.getType().toString());
conn.setDoOutput(false);
conn.setInstanceFollowRedirects(false);
conn.connect();
final String redirect = conn.getHeaderField("Location");
conn.disconnect();
//remove createparent
WebHdfsFileSystem.LOG.info("redirect = " + redirect);
String re = "&" + dnCreateParam + "=[^&]*";
String modified = redirect.replaceAll(re, "");
WebHdfsFileSystem.LOG.info("modified = " + modified);
//connect to datanode
conn = (HttpURLConnection)new URL(modified).openConnection();
conn.setRequestMethod(op.getType().toString());
conn.setDoOutput(op.getDoOutput());
conn.connect();
assertEquals(HttpServletResponse.SC_CREATED, conn.getResponseCode());
}
}
@Test
public void testAccess() throws IOException, InterruptedException {
Path p1 = new Path("/pathX");