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

Aaron Kimball commented on HADOOP-5958:
---------------------------------------

I'm not an interface fan myself. But the existing {{DF}} class already uses 
{{Shell}} as an abstract base class. That means we'd need to put most of the 
current DF's logic into some other class that extends Shell; maybe a private 
inner class?

+1 for a factory method.

> Use JDK 1.6 File APIs in DF.java wherever possible
> --------------------------------------------------
>
>                 Key: HADOOP-5958
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5958
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs
>            Reporter: Devaraj Das
>             Fix For: 0.21.0
>
>
> JDK 1.6 has File APIs like File.getFreeSpace() which should be used instead 
> of spawning a command process for getting the various disk/partition related 
> attributes. This would avoid spikes in memory consumption by tasks when 
> things like LocalDirAllocator is used for creating paths on the filesystem.

-- 
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