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

Jinglun commented on HDFS-14739:
--------------------------------

{quote}In {{TestDisableNameservices}} now both {{testWithoutDisabling}} and 
{{testDisabling}} seems almost same after the change.

The reason being {{dirns0}} is name of mount and also name in the default 
namespace. May be we can change a bit for {{testWithoutDisabling}} by either 
adding a directory in the default namespace, which is not a mount entry or 
changing the name of the directory in the default namespace.
{quote}
Hi [~ayushtkn], thanks your comments ! Very good suggestion ! I didn't 
understand this but now I know !

The listStatus() part test whether the nameservice is disabled successfully. We 
shouldn't be able to list the disabled nameservice in testDisabling(). So 
testWithoutDisabling and testDisabling need to be different. Before our fix the 
IOException makes them different,  after the fix we can make them different by 
adding a new directory, which is not a mount entry, to the default namespace.

 

Upload v06, pending jenkins.

> RBF: LS command for mount point shows wrong owner and permission information.
> -----------------------------------------------------------------------------
>
>                 Key: HDFS-14739
>                 URL: https://issues.apache.org/jira/browse/HDFS-14739
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: xuzq
>            Assignee: Jinglun
>            Priority: Major
>         Attachments: HDFS-14739-trunk-001.patch, HDFS-14739-trunk-002.patch, 
> HDFS-14739-trunk-003.patch, HDFS-14739-trunk-004.patch, 
> HDFS-14739-trunk-005.patch, HDFS-14739-trunk-006.patch, 
> image-2019-08-16-17-15-50-614.png, image-2019-08-16-17-16-00-863.png, 
> image-2019-08-16-17-16-34-325.png
>
>
> ||source||target namespace||destination||owner||group||permission||
> |/mnt|ns0|/mnt|mnt|mnt_group|755|
> |/mnt/test1|ns1|/mnt/test1|mnt_test1|mnt_test1_group|755|
> |/test1|ns1|/test1|test1|test1_group|755|
> When do getListing("/mnt"), the owner of  */mnt/test1* should be *mnt_test1* 
> instead of *test1* in result.
>  
> And if the mount table as blew, we should support getListing("/mnt") instead 
> of throw IOException when dfs.federation.router.default.nameservice.enable is 
> false.
> ||source||target namespace||destination||owner||group||permission||
> |/mnt/test1|ns0|/mnt/test1|test1|test1|755|
> |/mnt/test2|ns1|/mnt/test2|test2|test2|755|
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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