[
https://issues.apache.org/jira/browse/HDFS-14011?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16660134#comment-16660134
]
Yiqun Lin commented on HDFS-14011:
----------------------------------
[~ajisakaa], the patch almost looks good to me. Only one place:
*MockResolver.java*
{code}
public List<String> getMountPoints(String path) throws IOException {
+ // Mounts only supported under root level
+ if (!path.equals("/")) {
+ return null;
+ }
...
{code}
Here we might return empty list {{mounts}} rather than null. It will break some
places. Many places invoke this method and maybe not do null checking for the
returned list.
BTW, we don't need to update TestRouterRpcMultiDestination after this.
> RBF: Add more information to HdfsFileStatus for a mount point
> -------------------------------------------------------------
>
> Key: HDFS-14011
> URL: https://issues.apache.org/jira/browse/HDFS-14011
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Reporter: Akira Ajisaka
> Assignee: Akira Ajisaka
> Priority: Major
> Attachments: HDFS-14011.01.patch, HDFS-14011.02.patch,
> HDFS-14011.03.patch
>
>
> RouterClientProtocol#getMountPointStatus does not use information of the
> mount point, therefore, 'hdfs dfs -ls' to a directory including mount point
> returns the incorrect information.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
