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

Tsuyoshi Ozawa commented on HDFS-7863:
--------------------------------------

[~brahmareddy] thank you for updating a patch.

{code}
+   * @param src The given path
{code}

Please use lower case in the sentence - in the patch "The" should be "the". 
Also, we should describe more concrete comments here. I think "path name to be 
deleted" is looks instead of "The given path". Other parts look good to me.

> Missing description of parameter fsd in javadoc 
> ------------------------------------------------
>
>                 Key: HDFS-7863
>                 URL: https://issues.apache.org/jira/browse/HDFS-7863
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>            Reporter: Yongjun Zhang
>            Assignee: Brahma Reddy Battula
>            Priority: Minor
>         Attachments: HDFS-7863-002.patch, HDFS-7863-003.patch, HDFS-7863.patch
>
>
> HDFS-7573 did refactoring of delete() code. New parameter {{FSDirectory fsd}} 
> is added to resulted methods, but the javadoc is not updated.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to