[ 
https://issues.apache.org/jira/browse/HADOOP-14120?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Steve Loughran updated HADOOP-14120:
------------------------------------
       Resolution: Fixed
    Fix Version/s: 2.9.0
           Status: Resolved  (was: Patch Available)

committed! thanks!

(BTW, on the topic of testing, I did the -Dscale tests too. Why? To make sure 
the codepath related to multiple blocks uploaded from the same output stream 
got covered. Really we should be running those tests on any patch which touches 
BlockOutputStream)

> needless S3AFileSystem.setOptionalPutRequestParameters in 
> S3ABlockOutputStream putObject()
> ------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-14120
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14120
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 2.9.0
>            Reporter: Steve Loughran
>            Assignee: Yuanbo Liu
>            Priority: Minor
>             Fix For: 2.9.0
>
>         Attachments: HADOOP-14120.001.patch
>
>
> There's a call to {{S3AFileSystem.setOptionalPutRequestParameters()}} in {{ 
> S3ABlockOutputStream putObject()}}
> The put request has already been created by the FS; this call is only 
> superflous and potentially confusing.
> Proposed: cut it, make the {{setOptionalPutRequestParameters()}} method 
> private.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to