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

Steve Loughran commented on HADOOP-10643:
-----------------------------------------

Sumit -why did you cut a check from {{AbstractFileSystem}}? That could have 
implications elsewhere.

This should be retained, otherwise we've added a new way for things to fail, 
different error messages, etc.

Can you add a new protected method to validate the URI; something which does 
the check —and which your code overrides if it really, really doesn't want a 
port to be specified. That won't impact anything else.



> Add NativeS3Fs that delgates calls from FileContext apis to native s3 fs 
> implementation
> ---------------------------------------------------------------------------------------
>
>                 Key: HADOOP-10643
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10643
>             Project: Hadoop Common
>          Issue Type: New Feature
>          Components: fs/s3
>    Affects Versions: 2.4.0
>            Reporter: Sumit Kumar
>            Assignee: Sumit Kumar
>         Attachments: HADOOP-10643.patch
>
>
> The new set of file system related apis (FileContext/AbstractFileSystem) 
> already support local filesytem, hdfs, viewfs) however they don't support 
> s3n. This patch is to add that support using configurations like
> fs.AbstractFileSystem.s3n.impl = org.apache.hadoop.fs.s3native.NativeS3Fs
> This patch however doesn't provide a new implementation, instead relies on 
> DelegateToFileSystem abstract class to delegate all calls from FileContext 
> apis for s3n to the NativeS3FileSystem implementation.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to