[ https://issues.apache.org/jira/browse/HADOOP-8310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13260752#comment-13260752 ]
Aaron T. Myers commented on HADOOP-8310: ---------------------------------------- Thanks a lot for the review, John. I'm confident that the test failure of TestViewFsTrash is unrelated. I should also mention that I discovered this issue because without this fix one cannot view the conf page of a running MR2 job in a cluster whose fs.default.name is configured without a port. I manually verified that with this fix, one can get to the page just fine. I also ran the full HDFS test suite with this patch applied and everything passed. > FileContext#checkPath should handle URIs with no port > ----------------------------------------------------- > > Key: HADOOP-8310 > URL: https://issues.apache.org/jira/browse/HADOOP-8310 > Project: Hadoop Common > Issue Type: Bug > Components: fs > Affects Versions: 2.0.0 > Reporter: Aaron T. Myers > Assignee: Aaron T. Myers > Attachments: HADOOP-8310.patch > > > AbstractFileSystem#checkPath is used to verify that a given path is for the > same file system as represented by the AbstractFileSystem instance. > The original intent of the code was to allow for no port to be provided in > the checked path, if the default port was being used by the > AbstractFileSystem instance. However, before performing port handling, > AFS#checkPath compares the full URI authorities for equality. Since the URI > authority includes the port, the port handling code is never reached, and > thus valid paths may be erroneously considered invalid. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira