Hadoop Common
  1. Hadoop Common
  2. HADOOP-9241

DU refresh interval is not configurable

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.0.2-alpha
    • Fix Version/s: 2.3.0
    • Component/s: None
    • Labels:
      None
    • Release Note:
      The 'du' (disk usage command from Unix) script refresh monitor is now configurable in the same way as its 'df' counterpart, via the property 'fs.du.interval', the default of which is 10 minute (in ms).

      Description

      While the DF class's refresh interval is configurable, the DU's isn't. We should ensure both be configurable.

      1. HADOOP-9241.patch
        4 kB
        Harsh J
      2. HADOOP-9241.patch
        4 kB
        Harsh J

        Activity

        Hide
        Harsh J added a comment -

        Adds in a fs.du.interval config to DU akin to fs.df.interval from DF. DNs already use the conf-based API so no HDFS changes are needed at least.

        Show
        Harsh J added a comment - Adds in a fs.du.interval config to DU akin to fs.df.interval from DF. DNs already use the conf-based API so no HDFS changes are needed at least.
        Hide
        Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12566468/HADOOP-9241.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2091//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2091//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12566468/HADOOP-9241.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2091//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2091//console This message is automatically generated.
        Hide
        Harsh J added a comment -

        Since this is a trivial change and HDFS already uses the API whose implementation I've fixed here (keeping defaults the same):

        hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/BlockPoolSlice.java: this.dfsUsage = new DU(bpDir, conf);

        I went ahead and committed to both branch-2 and trunk.

        Show
        Harsh J added a comment - Since this is a trivial change and HDFS already uses the API whose implementation I've fixed here (keeping defaults the same): hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/fsdataset/impl/BlockPoolSlice.java: this.dfsUsage = new DU(bpDir, conf); I went ahead and committed to both branch-2 and trunk.
        Hide
        Hudson added a comment -

        Integrated in Hadoop-trunk-Commit #3285 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3285/)
        HADOOP-9241. DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129)

        Result = SUCCESS
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-trunk-Commit #3285 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3285/ ) HADOOP-9241 . DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129) Result = SUCCESS harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Yarn-trunk #110 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/110/)
        HADOOP-9241. DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129)

        Result = FAILURE
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-Yarn-trunk #110 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/110/ ) HADOOP-9241 . DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129) Result = FAILURE harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk #1299 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1299/)
        HADOOP-9241. DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129)

        Result = FAILURE
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #1299 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1299/ ) HADOOP-9241 . DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129) Result = FAILURE harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Mapreduce-trunk #1327 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1327/)
        HADOOP-9241. DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129)

        Result = FAILURE
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #1327 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1327/ ) HADOOP-9241 . DU refresh interval is not configurable. Contributed by Harsh J. (harsh) (Revision 1439129) Result = FAILURE harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439129 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Suresh Srinivas added a comment -

        Harsh J 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?

        -    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;
        
        Show
        Suresh Srinivas added a comment - Harsh J 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? - 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;
        Hide
        Harsh J added a comment -

        Sorry about that Suresh, will await a review for trivial ones in future as well. Indeed, I've made a big bad blooper thats not gonna help the DNs Fixing via HADOOP-9257 - apologies!

        Show
        Harsh J added a comment - Sorry about that Suresh, will await a review for trivial ones in future as well. Indeed, I've made a big bad blooper thats not gonna help the DNs Fixing via HADOOP-9257 - apologies!
        Hide
        Suresh Srinivas added a comment -

        Is it better to revert this and fix it?

        Show
        Suresh Srinivas added a comment - Is it better to revert this and fix it?
        Hide
        Harsh J added a comment -

        I suppose thats okay too - but I just submitted up a patch to HADOOP-9257. If you feel reverting is better, I'll get it done that way, let me know

        Show
        Harsh J added a comment - I suppose thats okay too - but I just submitted up a patch to HADOOP-9257 . If you feel reverting is better, I'll get it done that way, let me know
        Hide
        Tsz Wo Nicholas Sze added a comment -

        Harsh, please revert this for two reasons, (1) there is a bug and, more importantly, (2) it has not yet been reviewed.

        Also, please don't commit without review. Everyone has to follow the bylaws. If you feel that the current bylaws is inappropriate, start a discussion to change the bylaws first.

        Show
        Tsz Wo Nicholas Sze added a comment - Harsh, please revert this for two reasons, (1) there is a bug and, more importantly, (2) it has not yet been reviewed. Also, please don't commit without review. Everyone has to follow the bylaws. If you feel that the current bylaws is inappropriate, start a discussion to change the bylaws first.
        Hide
        Hudson added a comment -

        Integrated in Hadoop-trunk-Commit #3292 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3292/)
        Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750)
        Reverting HADOOP-9241. To be fixed and reviewed. (Revision 1439748)

        Result = SUCCESS
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml

        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-trunk-Commit #3292 (See https://builds.apache.org/job/Hadoop-trunk-Commit/3292/ ) Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750) Reverting HADOOP-9241 . To be fixed and reviewed. (Revision 1439748) Result = SUCCESS harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Harsh J added a comment -

        Thanks Nicholas; I have reverted HADOOP-9241 from trunk and branch-2. I will attach a proper patch now.

        Show
        Harsh J added a comment - Thanks Nicholas; I have reverted HADOOP-9241 from trunk and branch-2. I will attach a proper patch now.
        Hide
        Harsh J added a comment -

        Patch with 10m default preserved.

        Show
        Harsh J added a comment - Patch with 10m default preserved.
        Hide
        Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12566924/HADOOP-9241.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 1 new or modified test files.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The javadoc tool did not generate any warning messages.

        +1 eclipse:eclipse. The patch built with eclipse:eclipse.

        +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common.

        +1 contrib tests. The patch passed contrib unit tests.

        Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2110//testReport/
        Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2110//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12566924/HADOOP-9241.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2110//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2110//console This message is automatically generated.
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Yarn-trunk #111 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/111/)
        Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750)
        Reverting HADOOP-9241. To be fixed and reviewed. (Revision 1439748)

        Result = SUCCESS
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml

        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-Yarn-trunk #111 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/111/ ) Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750) Reverting HADOOP-9241 . To be fixed and reviewed. (Revision 1439748) Result = SUCCESS harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Hdfs-trunk #1300 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1300/)
        Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750)
        Reverting HADOOP-9241. To be fixed and reviewed. (Revision 1439748)

        Result = SUCCESS
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml

        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #1300 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1300/ ) Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750) Reverting HADOOP-9241 . To be fixed and reviewed. (Revision 1439748) Result = SUCCESS harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Mapreduce-trunk #1328 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1328/)
        Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750)
        Reverting HADOOP-9241. To be fixed and reviewed. (Revision 1439748)

        Result = SUCCESS
        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml

        harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748
        Files :

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #1328 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1328/ ) Revert HADOOP-9241 properly this time. Left the core-default.xml in previous commit. (Revision 1439750) Reverting HADOOP-9241 . To be fixed and reviewed. (Revision 1439748) Result = SUCCESS harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439750 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1439748 Files : /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Suresh Srinivas added a comment -

        +1 for the patch. Thanks for reworking the patch.

        Show
        Suresh Srinivas added a comment - +1 for the patch. Thanks for reworking the patch.
        Hide
        Hudson added a comment -

        SUCCESS: Integrated in Hadoop-trunk-Commit #4081 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4081/)
        HADOOP-9241. DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954)

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - SUCCESS: Integrated in Hadoop-trunk-Commit #4081 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4081/ ) HADOOP-9241 . DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Harsh J added a comment -

        Thanks! Committed to branch-2 and trunk.

        Show
        Harsh J added a comment - Thanks! Committed to branch-2 and trunk.
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk #1460 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1460/)
        HADOOP-9241. DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954)

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #1460 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1460/ ) HADOOP-9241 . DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk #1487 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1487/)
        HADOOP-9241. DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954)

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #1487 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1487/ ) HADOOP-9241 . DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Hide
        Hudson added a comment -

        FAILURE: Integrated in Hadoop-Yarn-trunk #271 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/271/)
        HADOOP-9241. DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954)

        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml
        • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java
        Show
        Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #271 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/271/ ) HADOOP-9241 . DU refresh interval is not configurable. (harsh) (harsh: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1502954 ) /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/CommonConfigurationKeysPublic.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/DU.java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/resources/core-default.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestDU.java

          People

          • Assignee:
            Harsh J
            Reporter:
            Harsh J
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development