[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15994220#comment-15994220 ] Weiwei Yang commented on YARN-6519: --- Thanks [~Naganarasimha] for all the way help :). > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Fix For: 2.9.0, 3.0.0-alpha3 > > Attachments: YARN-6519.001.patch, YARN-6519.002.patch, > YARN-6519-branch-2.001.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15992552#comment-15992552 ] Naganarasimha G R commented on YARN-6519: - thanks for the patch [~cheersyang], Test case failures are not related to the patch, committing it shortly ! > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch, > YARN-6519-branch-2.001.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15992326#comment-15992326 ] Hadoop QA commented on YARN-6519: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 14m 0s{color} | {color:blue} Docker mode activated. {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:green}+1{color} | {color:green} mvninstall {color} | {color:green} 7m 31s{color} | {color:green} branch-2 passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 33s{color} | {color:green} branch-2 passed with JDK v1.8.0_131 {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 35s{color} | {color:green} branch-2 passed with JDK v1.7.0_121 {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 27s{color} | {color:green} branch-2 passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 41s{color} | {color:green} branch-2 passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 17s{color} | {color:green} branch-2 passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 1m 22s{color} | {color:green} branch-2 passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 23s{color} | {color:green} branch-2 passed with JDK v1.8.0_131 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 26s{color} | {color:green} branch-2 passed with JDK v1.7.0_121 {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 33s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 33s{color} | {color:green} the patch passed with JDK v1.8.0_131 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 33s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 36s{color} | {color:green} the patch passed with JDK v1.7.0_121 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 36s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 26s{color} | {color:green} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 0 new + 265 unchanged - 3 fixed = 265 total (was 268) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 39s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 15s{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 28s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 21s{color} | {color:green} the patch passed with JDK v1.8.0_131 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 24s{color} | {color:green} the patch passed with JDK v1.7.0_121 {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 41m 12s{color} | {color:red} hadoop-yarn-server-resourcemanager in the patch failed with JDK v1.7.0_121. {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}115m 33s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | JDK v1.7.0_121 Failed junit tests | hadoop.yarn.server.resourcemanager.TestRMRestart | | | hadoop.yarn.server.resourcemanager.scheduler.capacity.TestCapacityScheduler | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:8515d35 | | JIRA Issue | YARN-6519 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12865865/YARN-6519-branch-2.001.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15992021#comment-15992021 ] Weiwei Yang commented on YARN-6519: --- Thank you [~Naganarasimha], just uploaded a patch for branch-2. > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch, > YARN-6519-branch-2.001.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15990931#comment-15990931 ] Hudson commented on YARN-6519: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11660 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/11660/]) YARN-6519. Fix warnings from Spotbugs in (naganarasimha_gr: rev 30fc5801966feb7f9bdd7d79db75acc595102913) * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CSQueueMetrics.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSQueueMetrics.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/FSSchedulerNode.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/QueueMetrics.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/AbstractYarnScheduler.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacitySchedulerQueueManager.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/monitor/capacity/ProportionalCapacityPreemptionPolicy.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/attempt/RMAppAttemptImpl.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/NodeType.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ApplicationMasterService.java * (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/attempt/RMAppAttemptMetrics.java > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hado
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15990908#comment-15990908 ] Naganarasimha G R commented on YARN-6519: - [~cheersyang], have committed this to trunk but cherry-pick seems to fail for branch-2. Can you please correct and provide a patch for branch-2 ? > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15990875#comment-15990875 ] Naganarasimha G R commented on YARN-6519: - Thanks for the updated patch [~cheersyang] and agree with all of your comments. Committing this patch ! > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15988893#comment-15988893 ] Weiwei Yang commented on YARN-6519: --- Thanks [~Naganarasimha], got a clean result this time, thanks for the help. Please let me know if this looks good to you. > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15988707#comment-15988707 ] Hadoop QA commented on YARN-6519: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 28s{color} | {color:blue} Docker mode activated. {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:green}+1{color} | {color:green} mvninstall {color} | {color:green} 17m 16s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 43s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 34s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 47s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 24s{color} | {color:green} trunk passed {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 1m 17s{color} | {color:red} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager in trunk has 8 extant Findbugs warnings. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 30s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 45s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 32s{color} | {color:green} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 0 new + 274 unchanged - 4 fixed = 274 total (was 278) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 40s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 18s{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 24s{color} | {color:green} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager generated 0 new + 0 unchanged - 8 fixed = 0 total (was 8) {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 22s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 43m 10s{color} | {color:green} hadoop-yarn-server-resourcemanager in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 23s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 71m 42s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:0ac17dc | | JIRA Issue | YARN-6519 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12864889/YARN-6519.002.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 891cfbb4674d 3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / cb672a4 | | Default Java | 1.8.0_121 | | findbugs | v3.1.0-RC1 | | findbugs | https://builds.apache.org/job/PreCommit-YARN-Build/15773/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html | | Test Results | https://builds.apache.org/job/PreCommit-YARN-Build/15773/testReport/ | | modules | C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15988700#comment-15988700 ] Hadoop QA commented on YARN-6519: - | (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: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:green}+1{color} | {color:green} mvninstall {color} | {color:green} 14m 19s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 33s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 29s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 36s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 17s{color} | {color:green} trunk passed {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 1m 4s{color} | {color:red} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager in trunk has 8 extant Findbugs warnings. {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 21s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 31s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 33s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 33s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 26s{color} | {color:green} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 0 new + 275 unchanged - 4 fixed = 275 total (was 279) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 35s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 15s{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 9s{color} | {color:green} hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager generated 0 new + 0 unchanged - 8 fixed = 0 total (was 8) {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 18s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 38m 32s{color} | {color:red} hadoop-yarn-server-resourcemanager in the patch failed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 21s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 61m 54s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.yarn.server.resourcemanager.TestRMRestart | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:0ac17dc | | JIRA Issue | YARN-6519 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12864889/YARN-6519.002.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux 7114973033a3 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20 09:57:27 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / cb672a4 | | Default Java | 1.8.0_121 | | findbugs | v3.1.0-RC1 | | findbugs | https://builds.apache.org/job/PreCommit-YARN-Build/15774/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html | | unit | https://builds.apache.org/job/PreCommit-YARN-Build/15774/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-res
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15988127#comment-15988127 ] Naganarasimha G R commented on YARN-6519: - [~cheersyang] Ohh sorry might be i had missed to trigger this manually ! any way it should run now after you resubmit if not by end of my time today will manually trigger the patch > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15988055#comment-15988055 ] Weiwei Yang commented on YARN-6519: --- Got jenkins result for the other 2 tasks, not sure why not on this one. Just re-submit the patch again to see if gets a better luck this time. > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15984978#comment-15984978 ] Weiwei Yang commented on YARN-6519: --- Thanks [~Naganarasimha] for your help ! > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch, YARN-6519.002.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15982320#comment-15982320 ] Weiwei Yang commented on YARN-6519: --- Hi [~Naganarasimha] Thanks for the review bq. NodeType ln no 29, no need to change the access specifier of the constructor Intellij gives a warning {{Modifier private is redundant for enum constructors}} if I keep the private modifier. This is because [http://docs.oracle.com/javase/specs/jls/se8/html/jls-8.html#jls-8.9.2] says {noformat} In an enum declaration, a constructor declaration with no access modifiers is private. {noformat} bq. EMPTY_CONTAINER_LIST can be private Sure, fixed. bq. QueueMetrics ln no 151, i was of the opinion that we use naming convention of field as its not exactly a string constant, thoughts? This is using name convention for {{final}} and {{static}} field, if not name it this way, we will get checkstyle warnings, similar names can be found a lot of places, such as {noformat} ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/CodecUtil.java:98: public static final Map DEFAULT_CODERS_MAP = ImmutableMap.of( ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/nativeio/NativeIO.java:465: private static final Map USER_ID_NAME_CACHE = ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/nativeio/NativeIO.java:468: private static final Map GROUP_ID_NAME_CACHE = ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/nativeio/NativeIO.java:702: private static final Map uidCache = ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/ObjectWritable.java:85: private static final Map> PRIMITIVE_NAMES = new HashMap>(); ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/WritableFactories.java:33: private static final Map CLASS_TO_FACTORY = ./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/RPC.java:190: private static final Map,RpcEngine> PROTOCOL_ENGINES {noformat} 4. is it possible for concurrent modification exception ? Good catch! Thanks! Fixed with following {code} preemptionCandidates.entrySet() .removeIf(candidate -> candidate.getValue() + 2 * maxWaitTime < currentTime); {code} 5. FSSchedulerNode ln no 157-165, i presume for removal we better not use entry set, Also a good catch, thanks. In order to fix the findbugs warning, we can still use entry set, just to iterate over this entry set {code} Iterator> iterator = resourcesPreemptedForApp.entrySet().iterator(); {code} Thanks for all the comments! Appreciate your review. > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) --
[jira] [Commented] (YARN-6519) Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager
[ https://issues.apache.org/jira/browse/YARN-6519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15981633#comment-15981633 ] Naganarasimha G R commented on YARN-6519: - Hi [~cheersyang], few nits in the patch as follows : # NodeType ln no 29, no need to change the access specifier of the constructor # ApplicationMasterService ln no 396, {{EMPTY_CONTAINER_LIST}} can be private and if required made default access in future #AbstractYarnScheduler ln no 135, {{EMPTY_CONTAINER_LIST}} as above can be private # QueueMetrics ln no 151, i was of the opinion that we use naming convention of field as its not exactly a string constant, thoughts? # ProportionalCapacityPreemptionPolicy ln no 311-315, as we are modifying by invoking {{preemptionCandidates.remove}} within the iterator whether is it possible for concurrent modification exception ? # FSSchedulerNode ln no 157-165, i presume for removal we better not use entry set, as there is Concurrent Modification Exception possible {code} public static void main(String args[]){ Map testMap = new HashMap<>(); testMap.put("1", "one"); testMap.put("2", "two"); testMap.put("3", "three"); testMap.put("4", "four"); for (Entry entry : testMap.entrySet()) { if (entry.getKey().equals("2")){ testMap.remove(entry.getKey()); } } } {code} > Fix warnings from Spotbugs in hadoop-yarn-server-resourcemanager > > > Key: YARN-6519 > URL: https://issues.apache.org/jira/browse/YARN-6519 > Project: Hadoop YARN > Issue Type: Bug > Components: resourcemanager >Reporter: Weiwei Yang >Assignee: Weiwei Yang > Labels: findbugs > Attachments: YARN-6519.001.patch > > > There is 8 findbugs warning in hadoop-yarn-server-timelineservice since > switched to spotbugs > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerQueueManager$1.compare(CSQueue, > CSQueue) incorrectly handles float value > # org.apache.hadoop.yarn.server.resourcemanager.scheduler.NodeType.index > field is public and mutable > # > org.apache.hadoop.yarn.server.resourcemanager.ApplicationMasterService.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.AbstractYarnScheduler.EMPTY_CONTAINER_LIST > is a mutable collection which should be package protected > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.QueueMetrics.queueMetrics > is a mutable collection > # > org.apache.hadoop.yarn.server.resourcemanager.monitor.capacity.ProportionalCapacityPreemptionPolicy.cleanupStaledPreemptionCandidates(long) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.rmapp.attempt.RMAppAttemptImpl.transferStateFromAttempt(RMAppAttempt) > makes inefficient use of keySet iterator instead of entrySet iterator > # > org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FSSchedulerNode.cleanupPreemptionList() > makes inefficient use of keySet iterator instead of entrySet iterator > See more from > [https://builds.apache.org/job/PreCommit-HADOOP-Build/12157/artifact/patchprocess/branch-findbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html] -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org