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

Raghu Angadi commented on HADOOP-1654:
--------------------------------------

Patch looks good. My only reservation is copyBytes() which takes conf.. Why not 
just require a buffer size. This is a utility function that is expected to be 
used in variety of places/contexts. conf is not always available. What do you 
think?


> IOUtils class
> -------------
>
>                 Key: HADOOP-1654
>                 URL: https://issues.apache.org/jira/browse/HADOOP-1654
>             Project: Hadoop
>          Issue Type: Improvement
>          Components: io
>    Affects Versions: 0.15.0
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>             Fix For: 0.15.0
>
>         Attachments: IOUtils_v1.0.patch, IOUtils_v1.1.patch, 
> IOUtils_v1.2.patch
>
>
> In the current situation, {{FileUtil}} class includes both file related and 
> io related functionality. This issue intends to separate the two. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to