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

Steve Loughran commented on HADOOP-12353:
-----------------------------------------

I'm looking at this. 

I see that it's trying to fix a problem —but I'm scared of making any changes 
to s3n these days. We think we are fixing bugs, but really we are moving them. 

So I'm an =0 here, not because there's anything wrong with the patch that I can 
see -only that I'd like s3n to be left alone and s3a being the place where we 
work on improving the s3 experience.

Looking at s3a, I don't see anything implemented in terms of retry policy. 
Would anyone be willing to start on doing something there for those initial 
connects, lists and seeks?

> S3 Native filesystem does not retry all connection failures
> -----------------------------------------------------------
>
>                 Key: HADOOP-12353
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12353
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs/s3
>    Affects Versions: 2.7.1
>            Reporter: Mariusz Strzelecki
>            Assignee: Mariusz Strzelecki
>            Priority: Minor
>         Attachments: HADOOP-12353.001.patch, HADOOP-12353.002.patch
>
>
> Current implementation of NativeS3FileSystem.java uses RetryProxy that 
> retries exceptions that may occur on network communication with S3 API, but 
> these exceptions must be exact instances of IOException:
> https://github.com/apache/hadoop/blob/master/hadoop-tools/hadoop-aws/src/main/java/org/apache/hadoop/fs/s3native/NativeS3FileSystem.java#L349
> Our tests show that HttpClient throws IOException subclasses which are not 
> handled by Proxy.
> Additionally, not all methods that call S3 API are listed to be handled, i.e. 
> storeEmptyFile and retrieveMetadata are missing.



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

Reply via email to