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

xuzq commented on HDFS-14739:
-----------------------------

Thanks [~ayushtkn] and [~elgoiri] for the comment.
{quote}Just to ignore exception at one place do we need this?
{quote}
I think we should ignore the IOException precisely like PathLocation is null or 
defaultNS disable. Other exception like multiple destinations, can we ignore it 
too? If it can ignore too, may be we don't need DestinationNullException.
{quote}Why there is change in contentsummary, Just because of change in 
exception?
{quote}
In the trunk code, it gets getContentSummary from the child if the child is not 
null. So getContentSummary should ignore the DestinationNullException too and 
try to get content summary from the child.
{quote}It looks like the main issue here is {{defaultNSEnable}}. Have you 
thought on just enabling the default namespace? Or even adding a / mount?
{quote}
In my production line, we disable the default namespace and don't have / mount.

 

> 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: xuzq
>            Priority: Major
>         Attachments: HDFS-14739-trunk-001.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.2#803003)

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