[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16175175#comment-16175175 ] Hanisha Koneru commented on HDFS-12371: --- Thank you [~kihwal] for reviewing and committing the patch and [~srikanth1184] for reporting the issue. > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > - > > Key: HDFS-12371 > URL: https://issues.apache.org/jira/browse/HDFS-12371 > Project: Hadoop HDFS > Issue Type: Bug > Components: metrics >Affects Versions: 2.7.1 >Reporter: Sai Nukavarapu >Assignee: Hanisha Koneru > Fix For: 2.9.0, 3.0.0-beta1, 2.8.3, 3.1.0 > > Attachments: HDFS-12371.001.patch, HDFS-12371.002.patch > > > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > Looking at the code, i see below description. > {noformat} > `BlockVerificationFailures` | Total number of verifications failures | > `BlocksVerified` | Total number of blocks verified | > {noformat} -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16174829#comment-16174829 ] Hudson commented on HDFS-12371: --- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #12935 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/12935/]) HDFS-12371. BlockVerificationFailures and BlocksVerified show up as 0 in (kihwal: rev 6bf921a5c3152a307b5c0903056d73ce07775a08) * (edit) hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/VolumeScanner.java > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > - > > Key: HDFS-12371 > URL: https://issues.apache.org/jira/browse/HDFS-12371 > Project: Hadoop HDFS > Issue Type: Bug > Components: metrics >Affects Versions: 2.7.1 >Reporter: Sai Nukavarapu >Assignee: Hanisha Koneru > Fix For: 2.9.0, 3.0.0-beta1, 2.8.3, 3.1.0 > > Attachments: HDFS-12371.001.patch, HDFS-12371.002.patch > > > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > Looking at the code, i see below description. > {noformat} > `BlockVerificationFailures` | Total number of verifications failures | > `BlocksVerified` | Total number of blocks verified | > {noformat} -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16174762#comment-16174762 ] Kihwal Lee commented on HDFS-12371: --- +1 lgtm > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > - > > Key: HDFS-12371 > URL: https://issues.apache.org/jira/browse/HDFS-12371 > Project: Hadoop HDFS > Issue Type: Bug > Components: metrics >Affects Versions: 2.7.1 >Reporter: Sai Nukavarapu >Assignee: Hanisha Koneru > Attachments: HDFS-12371.001.patch, HDFS-12371.002.patch > > > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > Looking at the code, i see below description. > {noformat} > `BlockVerificationFailures` | Total number of verifications failures | > `BlocksVerified` | Total number of blocks verified | > {noformat} -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16173607#comment-16173607 ] Hanisha Koneru commented on HDFS-12371: --- The test failures are unrelated. > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > - > > Key: HDFS-12371 > URL: https://issues.apache.org/jira/browse/HDFS-12371 > Project: Hadoop HDFS > Issue Type: Bug > Components: metrics >Affects Versions: 2.7.1 >Reporter: Sai Nukavarapu >Assignee: Hanisha Koneru > Attachments: HDFS-12371.001.patch, HDFS-12371.002.patch > > > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > Looking at the code, i see below description. > {noformat} > `BlockVerificationFailures` | Total number of verifications failures | > `BlocksVerified` | Total number of blocks verified | > {noformat} -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170871#comment-16170871 ] Hadoop QA commented on HDFS-12371: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 19s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 15m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 58s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 1m 2s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 58s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 44s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 1m 0s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 54s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 54s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 37s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 59s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 2m 2s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 43s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:red}-1{color} | {color:red} unit {color} | {color:red} 92m 57s{color} | {color:red} hadoop-hdfs in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 17s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}127m 29s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.server.namenode.metrics.TestNameNodeMetrics | | | hadoop.hdfs.TestLeaseRecoveryStriped | | | hadoop.hdfs.server.datanode.TestDataNodeVolumeFailure | | | hadoop.hdfs.TestReplication | | | hadoop.hdfs.TestReconstructStripedFile | | Timed out junit tests | org.apache.hadoop.hdfs.TestWriteReadStripedFile | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:71bbb86 | | JIRA Issue | HDFS-12371 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12887747/HDFS-12371.002.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 9f0b7d369651 3.13.0-117-generic #164-Ubuntu SMP Fri Apr 7 11:05:26 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 29dd551 | | Default Java | 1.8.0_144 | | findbugs | v3.1.0-RC1 | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/21200/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/21200/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/21200/console | | Powered by | Apache Yetus 0.6.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in
[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170581#comment-16170581 ] Kihwal Lee commented on HDFS-12371: --- Test failures are not related. They either pass when ran locally or are known issues. E.g. HDFS-12480 is addressing TestNameNodeMetrics. The block verification failure count was intended for actual checksum errors. I don't think we need to increment the failure count here. Other than that, the patch looks good. {code} if (block == null) { metrics.incrBlockVerificationFailures(); return -1; // block not found. } {code} > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > - > > Key: HDFS-12371 > URL: https://issues.apache.org/jira/browse/HDFS-12371 > Project: Hadoop HDFS > Issue Type: Bug > Components: metrics >Affects Versions: 2.7.1 >Reporter: Sai Nukavarapu >Assignee: Hanisha Koneru > Attachments: HDFS-12371.001.patch > > > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > Looking at the code, i see below description. > {noformat} > `BlockVerificationFailures` | Total number of verifications failures | > `BlocksVerified` | Total number of blocks verified | > {noformat} -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16170287#comment-16170287 ] Hadoop QA commented on HDFS-12371: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 13s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 13m 51s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 50s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 37s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 56s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 43s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 42s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 50s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 48s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 48s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 34s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 54s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 48s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 39s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:red}-1{color} | {color:red} unit {color} | {color:red} 87m 33s{color} | {color:red} hadoop-hdfs in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 16s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}113m 34s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.hdfs.server.namenode.metrics.TestNameNodeMetrics | | | hadoop.hdfs.TestDFSShellGenericOptions | | | hadoop.hdfs.TestParallelShortCircuitReadNoChecksum | | | hadoop.hdfs.TestLeaseRecoveryStriped | | Timed out junit tests | org.apache.hadoop.hdfs.TestWriteReadStripedFile | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:71bbb86 | | JIRA Issue | HDFS-12371 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12887457/HDFS-12371.001.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 745b94c3c220 3.13.0-123-generic #172-Ubuntu SMP Mon Jun 26 18:04:35 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 0f9af24 | | Default Java | 1.8.0_144 | | findbugs | v3.1.0-RC1 | | unit | https://builds.apache.org/job/PreCommit-HDFS-Build/21194/artifact/patchprocess/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HDFS-Build/21194/testReport/ | | modules | C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs | | Console output | https://builds.apache.org/job/PreCommit-HDFS-Build/21194/console | | Powered by | Apache Yetus 0.6.0-SNAPSHOT http://yetus.apache.org | This message was automatically generated. > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX >
[jira] [Commented] (HDFS-12371) "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX
[ https://issues.apache.org/jira/browse/HDFS-12371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16145482#comment-16145482 ] Kihwal Lee commented on HDFS-12371: --- {{BlockPoolSliceScanner}} used to call {{incrBlocksVerified()}}. HDFS-7430 reimplemented it as {{VolumeScanner}} and probably missed it. [~cmccabe_impala_fa3f], can you take a look? > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > - > > Key: HDFS-12371 > URL: https://issues.apache.org/jira/browse/HDFS-12371 > Project: Hadoop HDFS > Issue Type: Bug > Components: metrics >Affects Versions: 2.7.1 >Reporter: Sai Nukavarapu >Assignee: Hanisha Koneru > > "BlockVerificationFailures" and "BlocksVerified" show up as 0 in Datanode JMX > Looking at the code, i see below description. > {noformat} > `BlockVerificationFailures` | Total number of verifications failures | > `BlocksVerified` | Total number of blocks verified | > {noformat} -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org