[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13445947#comment-13445947 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Mapreduce-trunk #1182 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1182/]) HDFS-3837. Fix DataNode.recoverBlock findbugs warning. Contributed by Eli Collins (Revision 1379172) Result = SUCCESS eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1379172 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/dev-support/findbugsExcludeFile.xml > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Fix For: 2.2.0-alpha > > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt, > hdfs-3837.txt, hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13445892#comment-13445892 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Hdfs-trunk #1151 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk/1151/]) HDFS-3837. Fix DataNode.recoverBlock findbugs warning. Contributed by Eli Collins (Revision 1379172) Result = FAILURE eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1379172 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/dev-support/findbugsExcludeFile.xml > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Fix For: 2.2.0-alpha > > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt, > hdfs-3837.txt, hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13445248#comment-13445248 ] Hadoop QA commented on HDFS-3837: - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12543132/hdfs-3837.txt against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. 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. +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.server.blockmanagement.TestBlocksWithNotEnoughRacks org.apache.hadoop.hdfs.TestHftpDelegationToken +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/3125//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/3125//console This message is automatically generated. > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt, > hdfs-3837.txt, hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13444248#comment-13444248 ] Suresh Srinivas commented on HDFS-3837: --- Other than findbugs exclude, rest of the changes do not seem necessary or adds value. My preference is to undo it. However I do not feel strongly about this. +1 for the patch. > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt, > hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443914#comment-13443914 ] Hadoop QA commented on HDFS-3837: - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12542897/hdfs-3837.txt against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. 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. +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.TestHftpDelegationToken org.apache.hadoop.hdfs.server.namenode.metrics.TestNameNodeMetrics org.apache.hadoop.hdfs.web.TestWebHDFS org.apache.hadoop.hdfs.TestPersistBlocks +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/3117//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/3117//console This message is automatically generated. > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt, > hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443827#comment-13443827 ] Eli Collins commented on HDFS-3837: --- I investigated some more and confirmed findbugs isn't searching back far enough for the common subclass. Eg if I swap variables in the equals call I get: {noformat} org.apache.hadoop.hdfs.protocol.DatanodeInfo.equals(Object) used to determine equality org.apache.hadoop.hdfs.server.common.JspHelper$NodeRecord.equals(Object) used to determine equality org.apache.hadoop.hdfs.server.blockmanagement.DatanodeDescriptor.equals(Object) used to determine equality At DataNode.java:[line 1871] {noformat} It stops at DatanodeDescriptor#equals even though this calls super.equals (DatanodeInfo) which calls super.equals (DatanodeID). Just like the current warning stops at DatanodeRegistration#equals which calls super.equals (DatanodeID). It would be better (and findbugs wouldn't choke) if the various classes that extend DatanodeID have a member instead. I looked at this for HDFS-3237 and it required a ton of changes that probably aren't worth it. Given this I'll update the patch per your suggestion Surresh to ignore the warning in DataNode#recoverBlock. > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt, > hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443387#comment-13443387 ] Hadoop QA commented on HDFS-3837: - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12542780/hdfs-3837.txt against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. 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. +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 failed these unit tests in hadoop-hdfs-project/hadoop-hdfs: org.apache.hadoop.hdfs.TestHftpDelegationToken org.apache.hadoop.hdfs.server.namenode.metrics.TestNameNodeMetrics +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HDFS-Build/3108//testReport/ Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/3108//console This message is automatically generated. > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443286#comment-13443286 ] Suresh Srinivas commented on HDFS-3837: --- If this is a findbugs issue, why not just add this to findbugs exclude? > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt, hdfs-3837.txt, hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13443113#comment-13443113 ] Suresh Srinivas commented on HDFS-3837: --- Seems to me the findbugs is not fixed by the new patch or is it Jenkins error. Fixing this issue quickly will help. Currently all Jenkins reports have findbugs -1 for precommit tests. {noformat} Call to equals() comparing different types in org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) Bug type EC_UNRELATED_TYPES (click for details) In class org.apache.hadoop.hdfs.server.datanode.DataNode In method org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) Actual type org.apache.hadoop.hdfs.protocol.DatanodeInfo Expected org.apache.hadoop.hdfs.server.protocol.DatanodeRegistration Value loaded from id Value loaded from bpReg org.apache.hadoop.hdfs.server.protocol.DatanodeRegistration.equals(Object) used to determine equality At DataNode.java:[line 1869] {noformat} > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt, hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- 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
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13440791#comment-13440791 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Mapreduce-trunk-Commit #2659 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/2659/]) Revert HDFS-3837. Fix DataNode.recoverBlock findbugs warning. (Revision 1376701) Result = FAILURE eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1376701 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/protocol/DatanodeRegistration.java > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Fix For: 2.2.0-alpha > > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13440677#comment-13440677 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Common-trunk-Commit #2631 (See [https://builds.apache.org/job/Hadoop-Common-trunk-Commit/2631/]) Revert HDFS-3837. Fix DataNode.recoverBlock findbugs warning. (Revision 1376701) Result = SUCCESS eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1376701 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/protocol/DatanodeRegistration.java > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Fix For: 2.2.0-alpha > > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13440671#comment-13440671 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Hdfs-trunk-Commit #2695 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2695/]) Revert HDFS-3837. Fix DataNode.recoverBlock findbugs warning. (Revision 1376701) Result = SUCCESS eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1376701 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/protocol/DatanodeRegistration.java > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Fix For: 2.2.0-alpha > > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13440665#comment-13440665 ] Eli Collins commented on HDFS-3837: --- Re-opening this one, reverted the previous patch will post a new one. > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Fix For: 2.2.0-alpha > > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13440252#comment-13440252 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Hdfs-trunk #1143 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk/1143/]) HDFS-3837. Fix DataNode.recoverBlock findbugs warning. Contributed by Eli Collins (Revision 1376129) Result = FAILURE eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1376129 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/protocol/DatanodeRegistration.java > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Fix For: 2.2.0-alpha > > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13439700#comment-13439700 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Mapreduce-trunk-Commit #2648 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/2648/]) HDFS-3837. Fix DataNode.recoverBlock findbugs warning. Contributed by Eli Collins (Revision 1376129) Result = FAILURE eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1376129 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/protocol/DatanodeRegistration.java > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13439658#comment-13439658 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Hdfs-trunk-Commit #2682 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2682/]) HDFS-3837. Fix DataNode.recoverBlock findbugs warning. Contributed by Eli Collins (Revision 1376129) Result = SUCCESS eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1376129 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/protocol/DatanodeRegistration.java > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13439656#comment-13439656 ] Hudson commented on HDFS-3837: -- Integrated in Hadoop-Common-trunk-Commit #2618 (See [https://builds.apache.org/job/Hadoop-Common-trunk-Commit/2618/]) HDFS-3837. Fix DataNode.recoverBlock findbugs warning. Contributed by Eli Collins (Revision 1376129) Result = SUCCESS eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1376129 Files : * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt * /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/protocol/DatanodeRegistration.java > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HDFS-3837) Fix DataNode.recoverBlock findbugs warning
[ https://issues.apache.org/jira/browse/HDFS-3837?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13439641#comment-13439641 ] Suresh Srinivas commented on HDFS-3837: --- I had created HDFS-3836 for this. +1 for the patch. > Fix DataNode.recoverBlock findbugs warning > -- > > Key: HDFS-3837 > URL: https://issues.apache.org/jira/browse/HDFS-3837 > Project: Hadoop HDFS > Issue Type: Bug >Affects Versions: 2.0.0-alpha >Reporter: Eli Collins >Assignee: Eli Collins > Attachments: hdfs-3837.txt > > > HDFS-2686 introduced the following findbugs warning: > {noformat} > Call to equals() comparing different types in > org.apache.hadoop.hdfs.server.datanode.DataNode.recoverBlock(BlockRecoveryCommand$RecoveringBlock) > {noformat} > Both are using DatanodeID#equals but it's a different method because > DNR#equals overrides equals for some reason (doesn't change behavior). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira