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

John Zhuge commented on HDFS-11561:
-----------------------------------

Great work in patch 002. Just a few nitpicks.

index.md
* {{/user/foo.bar}} should be {{/user/foo/usr}} in the last curl command
* Wrap HTTP URL with single quotes in a all curl commands, e.g., {{curl 
http://...}} => {{curl 'http://...'}}. Otherwise, bash will interpret the {{&}} 
character.

TestHttpFSServer.java
* 218: Please revert if it is just coding style change
* 222: Please revert if it is just coding style change
* testHdfsRead: Has testOpenOffsetLength covered this case already?

> Mistake in httpfs doc file
> --------------------------
>
>                 Key: HDFS-11561
>                 URL: https://issues.apache.org/jira/browse/HDFS-11561
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: documentation, httpfs
>            Reporter: Yuanbo Liu
>            Assignee: Yuanbo Liu
>            Priority: Trivial
>         Attachments: HDFS-11561.001.patch, HDFS-11561.002.patch
>
>
> In the httpfs index.md, there is a sentence
> {quote}
> curl http://httpfs-host:14000/webhdfs/v1/user/foo?op=list` returns the 
> contents of ...
> {quote}
> which is wrong, the right operation is LISTSTATUS, not LIST.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to