[ 
https://issues.apache.org/jira/browse/HDFS-12564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16234398#comment-16234398
 ] 

Xiaoyu Yao commented on HDFS-12564:
-----------------------------------

Thanks [~tasanuma0829]. The patch looks good to me overall, here are a few 
comments:

Distcp.md.vm

Line 423: suggest adding a separate section and put the content(links) under it.

"#H3 Secure Copy over the wire with distcp"


ServerSetup.md.vm

This page is for HTTPFS. To avoid confusion, I would suggest we add a detailed 
ssl-client.xml example instead of linking it to 
Swebhdfs document. 


Webhdfs.md

Line 161: /etc/hadoop/hdfs-site.xml has a configuration key to enable secure 
http, 
i.e., dfs.http.policy=HTTPS_ONLY

Also note that dfs.http.policy is not for swebhdfs only. This will also affect 
all the HTTP endpoints of HDFS such as the NN, DN WebUI, JMX, QJM.
Line 198: suggest give a full path: ssl-client.xml -> 
/etc/hadoop/ssl-client.xml 


We also need to document settings for the server side settings, e.g., 
ssl-server.xml. 

> Add the documents of swebhdfs configurations on the client side
> ---------------------------------------------------------------
>
>                 Key: HDFS-12564
>                 URL: https://issues.apache.org/jira/browse/HDFS-12564
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: documentation, webhdfs
>            Reporter: Takanobu Asanuma
>            Assignee: Takanobu Asanuma
>            Priority: Major
>         Attachments: HDFS-12564.1.patch, HDFS-12564.2.patch
>
>
> Documentation does not cover the swebhdfs configurations on the client side. 
> We can reuse the hftp/hsftp documents which was removed from Hadoop-3.0 in 
> HDFS-5570, HDFS-9640.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to