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

Íñigo Goiri commented on HDFS-14085:
------------------------------------

Thanks for splitting the tests; looks good.

We should fix those checkstyles; I guess it comes from not wanting to add 
spaces to the whole class.
I think we could have an After method to cleanup instead of having to do the 
finally.

Regarding the addition of the "/" it makes sense but I'm a little concerned 
about the test coverage.
We were going through that part of the code and we were able to list the paths 
with no issues?
Are we covering this case now?

> RBF: LS command for root shows wrong owner and permission information.
> ----------------------------------------------------------------------
>
>                 Key: HDFS-14085
>                 URL: https://issues.apache.org/jira/browse/HDFS-14085
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Ayush Saxena
>            Assignee: Ayush Saxena
>            Priority: Major
>         Attachments: HDFS-14085-HDFS-13891-01.patch, 
> HDFS-14085-HDFS-13891-02.patch
>
>
> The LS command for / lists all the mount entries but the permission displayed 
> is the default permission (777) and the owner and group info same as that of 
> the user calling it; Which actually should be the same as that of the 
> destination of the mount point.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
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