Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0.0-alpha2
    • Component/s: None
    • Labels:
      None

      Description

      This is a sub task of HDFS-7674. It adds time metric for ec decode work.

      1. HDFS-8410-001.patch
        6 kB
        Li Bo
      2. HDFS-8410-002.patch
        4 kB
        Li Bo
      3. HDFS-8410-003.patch
        4 kB
        SammiChen
      4. HDFS-8410-004.patch
        5 kB
        SammiChen
      5. HDFS-8410-005.patch
        6 kB
        SammiChen

        Issue Links

          Activity

          Hide
          libo-intel Li Bo added a comment -

          The patch is based on HDFS-8449

          Show
          libo-intel Li Bo added a comment - The patch is based on HDFS-8449
          Hide
          walter.k.su Walter Su added a comment -

          1. three metrics are redundant. 2. Is average time better?

          Show
          walter.k.su Walter Su added a comment - 1. three metrics are redundant. 2. Is average time better?
          Hide
          libo-intel Li Bo added a comment -

          Patch 002 reduces metrics number grom 3 to 2. The time metrics make user know how much time spent in encoding/decoding work for every datanode.

          Show
          libo-intel Li Bo added a comment - Patch 002 reduces metrics number grom 3 to 2. The time metrics make user know how much time spent in encoding/decoding work for every datanode.
          Hide
          Sammi SammiChen added a comment -

          1. Rebase patch against trunk
          2. Improve test function

          Show
          Sammi SammiChen added a comment - 1. Rebase patch against trunk 2. Improve test function
          Hide
          hadoopqa Hadoop QA added a comment -
          -1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 20s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
          +1 mvninstall 7m 14s trunk passed
          +1 compile 0m 45s trunk passed
          +1 checkstyle 0m 28s trunk passed
          +1 mvnsite 0m 53s trunk passed
          +1 mvneclipse 0m 13s trunk passed
          +1 findbugs 1m 46s trunk passed
          +1 javadoc 0m 38s trunk passed
          +1 mvninstall 0m 46s the patch passed
          +1 compile 0m 42s the patch passed
          +1 javac 0m 42s the patch passed
          -0 checkstyle 0m 25s hadoop-hdfs-project/hadoop-hdfs: The patch generated 2 new + 64 unchanged - 0 fixed = 66 total (was 64)
          +1 mvnsite 0m 50s the patch passed
          +1 mvneclipse 0m 10s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 1m 51s the patch passed
          +1 javadoc 0m 37s the patch passed
          -1 unit 58m 29s hadoop-hdfs in the patch failed.
          +1 asflicense 0m 20s The patch does not generate ASF License warnings.
          77m 43s



          Reason Tests
          Failed junit tests hadoop.hdfs.TestDistributedFileSystem



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:9560f25
          JIRA Issue HDFS-8410
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12833884/HDFS-8410-003.patch
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux 5b901de40549 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision trunk / c023c74
          Default Java 1.8.0_101
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/17197/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
          unit https://builds.apache.org/job/PreCommit-HDFS-Build/17197/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
          Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/17197/testReport/
          modules C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs
          Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17197/console
          Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 20s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. +1 mvninstall 7m 14s trunk passed +1 compile 0m 45s trunk passed +1 checkstyle 0m 28s trunk passed +1 mvnsite 0m 53s trunk passed +1 mvneclipse 0m 13s trunk passed +1 findbugs 1m 46s trunk passed +1 javadoc 0m 38s trunk passed +1 mvninstall 0m 46s the patch passed +1 compile 0m 42s the patch passed +1 javac 0m 42s the patch passed -0 checkstyle 0m 25s hadoop-hdfs-project/hadoop-hdfs: The patch generated 2 new + 64 unchanged - 0 fixed = 66 total (was 64) +1 mvnsite 0m 50s the patch passed +1 mvneclipse 0m 10s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 51s the patch passed +1 javadoc 0m 37s the patch passed -1 unit 58m 29s hadoop-hdfs in the patch failed. +1 asflicense 0m 20s The patch does not generate ASF License warnings. 77m 43s Reason Tests Failed junit tests hadoop.hdfs.TestDistributedFileSystem Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Issue HDFS-8410 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12833884/HDFS-8410-003.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 5b901de40549 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / c023c74 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/17197/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt unit https://builds.apache.org/job/PreCommit-HDFS-Build/17197/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/17197/testReport/ modules C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17197/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          andrew.wang Andrew Wang added a comment -

          Hi SammiChen thanks for working on this. One question, ECEncodingTime isn't used right now, can we remove it? Also it might be better to use GenericTestUtils.waitFor in the test to wait for the metric.

          Show
          andrew.wang Andrew Wang added a comment - Hi SammiChen thanks for working on this. One question, ECEncodingTime isn't used right now, can we remove it? Also it might be better to use GenericTestUtils.waitFor in the test to wait for the metric.
          Hide
          Sammi SammiChen added a comment -

          Thanks Andrew, for taking time review the patch and give detail suggestions! I uploaded a new patch. The difference between this version and last version patch are,
          1. remove the ECEncodingTime metric
          2. Use GenericTestUtils.waitFor to wait for reconstruction task finish
          3 Refactor test testEcTasks, use GenericTestUtils.waitFor instead of current wait logic

          Show
          Sammi SammiChen added a comment - Thanks Andrew, for taking time review the patch and give detail suggestions! I uploaded a new patch. The difference between this version and last version patch are, 1. remove the ECEncodingTime metric 2. Use GenericTestUtils.waitFor to wait for reconstruction task finish 3 Refactor test testEcTasks, use GenericTestUtils.waitFor instead of current wait logic
          Hide
          hadoopqa Hadoop QA added a comment -
          +1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 20s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
          +1 mvninstall 14m 3s trunk passed
          +1 compile 0m 48s trunk passed
          +1 checkstyle 0m 27s trunk passed
          +1 mvnsite 0m 56s trunk passed
          +1 mvneclipse 0m 12s trunk passed
          +1 findbugs 1m 49s trunk passed
          +1 javadoc 0m 41s trunk passed
          +1 mvninstall 0m 48s the patch passed
          +1 compile 0m 45s the patch passed
          +1 javac 0m 45s the patch passed
          -0 checkstyle 0m 25s hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 64 unchanged - 0 fixed = 65 total (was 64)
          +1 mvnsite 0m 50s the patch passed
          +1 mvneclipse 0m 10s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 1m 59s the patch passed
          +1 javadoc 0m 39s the patch passed
          +1 unit 60m 1s hadoop-hdfs in the patch passed.
          +1 asflicense 0m 18s The patch does not generate ASF License warnings.
          86m 28s



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:9560f25
          JIRA Issue HDFS-8410
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12834114/HDFS-8410-004.patch
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux cea6bb9c1b47 3.13.0-92-generic #139-Ubuntu SMP Tue Jun 28 20:42:26 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision trunk / c5573e6
          Default Java 1.8.0_101
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/17214/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
          Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/17214/testReport/
          modules C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs
          Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17214/console
          Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - +1 overall Vote Subsystem Runtime Comment 0 reexec 0m 20s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. +1 mvninstall 14m 3s trunk passed +1 compile 0m 48s trunk passed +1 checkstyle 0m 27s trunk passed +1 mvnsite 0m 56s trunk passed +1 mvneclipse 0m 12s trunk passed +1 findbugs 1m 49s trunk passed +1 javadoc 0m 41s trunk passed +1 mvninstall 0m 48s the patch passed +1 compile 0m 45s the patch passed +1 javac 0m 45s the patch passed -0 checkstyle 0m 25s hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 64 unchanged - 0 fixed = 65 total (was 64) +1 mvnsite 0m 50s the patch passed +1 mvneclipse 0m 10s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 59s the patch passed +1 javadoc 0m 39s the patch passed +1 unit 60m 1s hadoop-hdfs in the patch passed. +1 asflicense 0m 18s The patch does not generate ASF License warnings. 86m 28s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Issue HDFS-8410 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12834114/HDFS-8410-004.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux cea6bb9c1b47 3.13.0-92-generic #139-Ubuntu SMP Tue Jun 28 20:42:26 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / c5573e6 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/17214/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/17214/testReport/ modules C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17214/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          andrew.wang Andrew Wang added a comment -

          Sorry I didn't mention this in the previous review, but it'd be mildly better to use "final" for the two vars, as well as Time.monotonicNow() which is our wrapper for nanotime. +1 pending this though, thanks Sammi.

          Show
          andrew.wang Andrew Wang added a comment - Sorry I didn't mention this in the previous review, but it'd be mildly better to use "final" for the two vars, as well as Time.monotonicNow() which is our wrapper for nanotime. +1 pending this though, thanks Sammi.
          Hide
          Sammi SammiChen added a comment -

          Hi Andrew, thanks for the comments. Yea, It's better to use "final" for the new variable. Regarding using Time.monotonicNow() function, it seems this function returns the millisecond value instead of nanoseconds value. Given this metric is called "Nanoseconds spent by decoding tasks", I'm not sure if you are suggesting change this metric definition to "Milliseconds" something. I'd like to known your opinion.

          Show
          Sammi SammiChen added a comment - Hi Andrew, thanks for the comments. Yea, It's better to use "final" for the new variable. Regarding using Time.monotonicNow() function, it seems this function returns the millisecond value instead of nanoseconds value. Given this metric is called "Nanoseconds spent by decoding tasks", I'm not sure if you are suggesting change this metric definition to "Milliseconds" something. I'd like to known your opinion.
          Hide
          andrew.wang Andrew Wang added a comment -

          Good point. Do you have a sense of the order of magnitude of these values? Most of the other DatanodeMetrics use milliseconds, so that'd be good for consistency.

          Show
          andrew.wang Andrew Wang added a comment - Good point. Do you have a sense of the order of magnitude of these values? Most of the other DatanodeMetrics use milliseconds, so that'd be good for consistency.
          Hide
          Sammi SammiChen added a comment -

          It costs about 2 milliseconds in my desktop to decode a strip group with 6 blocks, each block is 64k. This decoding time merely depends on how fast the CPU is. It's "Intel(R) Core(TM) i5-4460 CPU @ 3.20GHz" with 4 cores in my desktop, not the leading edge CPU model. Given that CPU is becoming more and more powerful, I think it is not safe to use millisecond granularity to record one time decoding time. We can choose between nanosecond or microsecond. I would prefer nanosecond for one reason. it can be directly get through System.nanoTime(). If microsecond is used, there is one extra division of 1000. That's not good from performance point of view. And a long number can host nanoseconds, representing hundreds of years. So there is not going to a overflow quickly.

          Show
          Sammi SammiChen added a comment - It costs about 2 milliseconds in my desktop to decode a strip group with 6 blocks, each block is 64k. This decoding time merely depends on how fast the CPU is. It's "Intel(R) Core(TM) i5-4460 CPU @ 3.20GHz" with 4 cores in my desktop, not the leading edge CPU model. Given that CPU is becoming more and more powerful, I think it is not safe to use millisecond granularity to record one time decoding time. We can choose between nanosecond or microsecond. I would prefer nanosecond for one reason. it can be directly get through System.nanoTime() . If microsecond is used, there is one extra division of 1000. That's not good from performance point of view. And a long number can host nanoseconds, representing hundreds of years. So there is not going to a overflow quickly.
          Hide
          Sammi SammiChen added a comment -

          1. add "final" to metric variable
          2. the previous checkstyle issue is about "the variable should has get/set function". Leave it alone.

          Show
          Sammi SammiChen added a comment - 1. add "final" to metric variable 2. the previous checkstyle issue is about "the variable should has get/set function". Leave it alone.
          Hide
          hadoopqa Hadoop QA added a comment -
          -1 overall



          Vote Subsystem Runtime Comment
          0 reexec 0m 18s Docker mode activated.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
          +1 mvninstall 7m 22s trunk passed
          +1 compile 0m 48s trunk passed
          +1 checkstyle 0m 26s trunk passed
          +1 mvnsite 1m 0s trunk passed
          +1 mvneclipse 0m 13s trunk passed
          +1 findbugs 1m 44s trunk passed
          +1 javadoc 0m 38s trunk passed
          +1 mvninstall 0m 47s the patch passed
          +1 compile 0m 44s the patch passed
          +1 javac 0m 44s the patch passed
          -0 checkstyle 0m 24s hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 63 unchanged - 0 fixed = 64 total (was 63)
          +1 mvnsite 0m 59s the patch passed
          +1 mvneclipse 0m 12s the patch passed
          +1 whitespace 0m 0s The patch has no whitespace issues.
          +1 findbugs 2m 10s the patch passed
          +1 javadoc 0m 44s the patch passed
          -1 unit 78m 30s hadoop-hdfs in the patch failed.
          +1 asflicense 0m 22s The patch does not generate ASF License warnings.
          98m 51s



          Reason Tests
          Failed junit tests hadoop.hdfs.server.datanode.fsdataset.impl.TestLazyPersistReplicaRecovery



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:9560f25
          JIRA Issue HDFS-8410
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12834325/HDFS-8410-005.patch
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux 5c07a2ec3f04 3.13.0-96-generic #143-Ubuntu SMP Mon Aug 29 20:15:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
          Build tool maven
          Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
          git revision trunk / 73504b1
          Default Java 1.8.0_101
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/17234/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
          unit https://builds.apache.org/job/PreCommit-HDFS-Build/17234/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
          Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/17234/testReport/
          modules C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs
          Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17234/console
          Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 18s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. +1 mvninstall 7m 22s trunk passed +1 compile 0m 48s trunk passed +1 checkstyle 0m 26s trunk passed +1 mvnsite 1m 0s trunk passed +1 mvneclipse 0m 13s trunk passed +1 findbugs 1m 44s trunk passed +1 javadoc 0m 38s trunk passed +1 mvninstall 0m 47s the patch passed +1 compile 0m 44s the patch passed +1 javac 0m 44s the patch passed -0 checkstyle 0m 24s hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 63 unchanged - 0 fixed = 64 total (was 63) +1 mvnsite 0m 59s the patch passed +1 mvneclipse 0m 12s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 2m 10s the patch passed +1 javadoc 0m 44s the patch passed -1 unit 78m 30s hadoop-hdfs in the patch failed. +1 asflicense 0m 22s The patch does not generate ASF License warnings. 98m 51s Reason Tests Failed junit tests hadoop.hdfs.server.datanode.fsdataset.impl.TestLazyPersistReplicaRecovery Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Issue HDFS-8410 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12834325/HDFS-8410-005.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 5c07a2ec3f04 3.13.0-96-generic #143-Ubuntu SMP Mon Aug 29 20:15:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 73504b1 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-HDFS-Build/17234/artifact/patchprocess/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt unit https://builds.apache.org/job/PreCommit-HDFS-Build/17234/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt Test Results https://builds.apache.org/job/PreCommit-HDFS-Build/17234/testReport/ modules C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs Console output https://builds.apache.org/job/PreCommit-HDFS-Build/17234/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          Hide
          Sammi SammiChen added a comment -

          1. The failed test is irrelevant. The failure reason is "Adress already in use"
          2. The checkstyle issue is about "the variable should has get/set function". That's the not case since other variables don't has get/set function either.

          Show
          Sammi SammiChen added a comment - 1. The failed test is irrelevant. The failure reason is "Adress already in use" 2. The checkstyle issue is about "the variable should has get/set function". That's the not case since other variables don't has get/set function either.
          Hide
          andrew.wang Andrew Wang added a comment -

          Thanks Sammi, I've committed your patch to trunk. I noticed that start/end aren't "final" but that's not a big deal.

          Show
          andrew.wang Andrew Wang added a comment - Thanks Sammi, I've committed your patch to trunk. I noticed that start/end aren't "final" but that's not a big deal.
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10656 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10656/)
          HDFS-8410. Add computation time metrics to datanode for ECWorker. (wang: rev 61e30cf83ca78529603d9b4c6732418da7e4d0c8)

          • (edit) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/StripedBlockReconstructor.java
          • (edit) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/metrics/DataNodeMetrics.java
          • (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/datanode/TestDataNodeErasureCodingMetrics.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10656 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10656/ ) HDFS-8410 . Add computation time metrics to datanode for ECWorker. (wang: rev 61e30cf83ca78529603d9b4c6732418da7e4d0c8) (edit) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/StripedBlockReconstructor.java (edit) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/metrics/DataNodeMetrics.java (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/datanode/TestDataNodeErasureCodingMetrics.java
          Hide
          Sammi SammiChen added a comment -

          Thanks Andrew for review and commit the patch!

          Show
          Sammi SammiChen added a comment - Thanks Andrew for review and commit the patch!

            People

            • Assignee:
              Sammi SammiChen
              Reporter:
              libo-intel Li Bo
            • Votes:
              1 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development