[
https://issues.apache.org/jira/browse/HADOOP-8320?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14572521#comment-14572521
]
Vinayakumar B commented on HADOOP-8320:
---------------------------------------
[~vijaysbhat], thanks for the update on patch.
Two things need to be corrected.
# AbstractFileSystem doesn't support relative paths, Instead it supports only
fully qualified or paths starting with root '/'. Its okay untill relative path
is corrected already in FileContext before passing to AFS.
# Usage of {{NetUtils.getCanonicalUri(uri, getDefaultPort());}} was
intentionally not done, because there was no need of DNS replacement for the
host, which might be time consuming, instead just needed to replace default
port. Check javadoc in {{FileSystem#canonicalizeUri()}}
> FileSystem#checkPath and AbstractFileSystem#checkPath should share code
> -----------------------------------------------------------------------
>
> Key: HADOOP-8320
> URL: https://issues.apache.org/jira/browse/HADOOP-8320
> Project: Hadoop Common
> Issue Type: Improvement
> Components: fs
> Affects Versions: 2.0.0-alpha
> Reporter: Aaron T. Myers
> Assignee: Vijay Bhat
> Priority: Minor
> Labels: BB2015-05-TBR
> Attachments: HADOOP-8320.001.patch, HADOOP-8320.002.patch,
> HADOOP-8320.003.patch, HADOOP-8320.004.patch, HADOOP-8320.005.patch,
> HADOOP-8320.007.patch
>
>
> Per the discussion on HADOOP-8310, these two methods can be refactored to
> share some code.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
