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

Tsz Wo Nicholas Sze commented on HDFS-6438:
-------------------------------------------

Some more minor comments on the doc:
- In CREATESNAPSHOT, the snapshotname is an optional parameter so that we 
should add [ ] in the curl command example.
- In RENAMESNAPSHOT, it should be oldsnapshotname=<SNAPSHOTNAME> since the < > 
is used to enclose the variable type.  Both oldsnapshotname and snapshotname 
have the same type <SNAPSHOTNAME>.
- The HTTP Query Parameter Dictionary is in alphabetical order so that the new 
Snapshot Name and Old Snapshot Name tables should be relocated accordingly.

+1 once these have been addressed.

> DeleteSnapshot should be a DELETE request in WebHdfs
> ----------------------------------------------------
>
>                 Key: HDFS-6438
>                 URL: https://issues.apache.org/jira/browse/HDFS-6438
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: webhdfs
>    Affects Versions: 2.5.0
>            Reporter: Jing Zhao
>            Assignee: Jing Zhao
>         Attachments: HDFS-6438.000.patch, HDFS-6438.001.patch, 
> HDFS-6438.002.patch
>
>
> We add 3 snapshot related APIs into WebHdfs in HDFS-6432, among which 
> DeleteSnapshot should be a DELETE request instead of PUT. This jira aims to 
> fix this and update the WebHdfs document.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to