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

Uma Maheswara Rao G commented on HADOOP-8861:
---------------------------------------------

Thanks Amareshwari, for the patch.

Change make sense to me. Do you mind having a small test like 
TestLocalFileSystem#testSyncable in trunk.
Other than that, +1 for the patch.
                
> FSDataOutputStream.sync should call flush() if the underlying wrapped stream 
> is not Syncable
> --------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-8861
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8861
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 1.0.3
>            Reporter: Amareshwari Sriramadasu
>         Attachments: HADOOP-8861.patch
>
>
> Currently FSDataOutputStream.sync is a no-op if the wrapped stream is not 
> Syncable. Instead it should call flush() if the wrapped stream is not 
> syncable.
> This behavior is already present in trunk, but branch-1 does not have this.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to