[
https://issues.apache.org/jira/browse/HDFS-14739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913627#comment-16913627
]
Íñigo Goiri commented on HDFS-14739:
------------------------------------
I tried to go through the discussion but it was a little hard to follow so I
went for the patch itself.
I may have missed some of the discussion so excuse me if I say anything
redundant here.
Going through the patch:
* It looks like the main issue here is {{defaultNSEnable}}. Have you thought on
just enabling the default namespace? Or even adding a / mount?
* For the new exception, the javadoc should give an example, kind of hard to
get otherwise.
* We may want to add the path as a field to the exception.
* When we capture the exception in getListing(), we should do something with
it. Maybe log it. Ideally we should "check" those in the test.
* What is the fix in RouterClientProtocol#811?
> 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: [email protected]
For additional commands, e-mail: [email protected]
