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

Suresh Srinivas commented on HADOOP-9241:
-----------------------------------------

[~qwertymaniac] Even for trivial jiras, I suggest getting the code review done 
before committing the code. Such changes are easy and quick to review.

In this patch, did DU interval become 1 minute instead of 10 minutes?
{code}
-    this(path, 600000L);
-    //10 minutes default refresh interval
+    this(path, conf.getLong(CommonConfigurationKeys.FS_DU_INTERVAL_KEY,
+                CommonConfigurationKeys.FS_DU_INTERVAL_DEFAULT));


+  /** See <a href="{@docRoot}/../core-default.html">core-default.xml</a> */
+  public static final String  FS_DU_INTERVAL_KEY = "fs.du.interval";
+  /** Default value for FS_DU_INTERVAL_KEY */
+  public static final long    FS_DU_INTERVAL_DEFAULT = 60000;
{code}
                
> DU refresh interval is not configurable
> ---------------------------------------
>
>                 Key: HADOOP-9241
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9241
>             Project: Hadoop Common
>          Issue Type: Improvement
>    Affects Versions: 2.0.2-alpha
>            Reporter: Harsh J
>            Assignee: Harsh J
>            Priority: Trivial
>             Fix For: 2.0.3-alpha
>
>         Attachments: HADOOP-9241.patch
>
>
> While the {{DF}} class's refresh interval is configurable, the {{DU}}'s 
> isn't. We should ensure both be configurable.

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