[ 
https://issues.apache.org/jira/browse/YARN-10358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17161998#comment-17161998
 ] 

Masatake Iwasaki commented on YARN-10358:
-----------------------------------------

WMI_WRONG_MAP_ITERATOR is not reported for EnumMap after findbugs-3.0.2 (i.e. 
spotbugs). findbugs-3.0.1 is used under the hood in branch-2.10. We can not 
migrate to spotbugs since it does not support Java 7. We should filter the 
warnings out by excludeFilterFile.
 [https://sourceforge.net/p/findbugs/bugs/1422/]
 
[https://github.com/findbugsproject/findbugs/commit/4d41d9704295e4be15de892f116090cd096a97f6]

> Fix findbugs warnings in hadoop-yarn-project on branch-2.10
> -----------------------------------------------------------
>
>                 Key: YARN-10358
>                 URL: https://issues.apache.org/jira/browse/YARN-10358
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.10.0
>            Reporter: Masatake Iwasaki
>            Assignee: Masatake Iwasaki
>            Priority: Minor
>
> {noformat}
> $ find . -name findbugsXml.xml | xargs -n 1 
> /opt/findbugs-3.0.1/bin/convertXmlToText -longBugCodes
> M P WMI_WRONG_MAP_ITERATOR WMI: 
> org.apache.hadoop.yarn.state.StateMachineFactory.generateStateGraph(String) 
> makes inefficient use of keySet iterator instead of entrySet iterator  At 
> StateMachineFactory.java:[line 505]
> M P BX_UNBOXING_IMMEDIATELY_REBOXED Bx: Boxed value is unboxed and then 
> immediately reboxed in 
> org.apache.hadoop.yarn.server.timelineservice.storage.common.ColumnRWHelper.readResultsWithTimestamps(Result,
>  byte[], byte[], KeyConverter, ValueConverter, boolean)  At 
> ColumnRWHelper.java:[line 335]
> M D UC_USELESS_OBJECT UC: Useless object stored in variable 
> removedNullContainers of method 
> org.apache.hadoop.yarn.server.nodemanager.NodeStatusUpdaterImpl.removeOrTrackCompletedContainersFromContext(List)
>   At NodeStatusUpdaterImpl.java:[line 664]
> M P WMI_WRONG_MAP_ITERATOR WMI: 
> org.apache.hadoop.yarn.server.nodemanager.NodeStatusUpdaterImpl.removeVeryOldStoppedContainersFromCache()
>  makes inefficient use of keySet iterator instead of entrySet iterator  At 
> NodeStatusUpdaterImpl.java:[line 741]
> M P WMI_WRONG_MAP_ITERATOR WMI: 
> org.apache.hadoop.yarn.server.nodemanager.containermanager.localizer.ContainerLocalizer.createStatus()
>  makes inefficient use of keySet iterator instead of entrySet iterator  At 
> ContainerLocalizer.java:[line 359]
> M V MS_MUTABLE_COLLECTION_PKGPROTECT MS: 
> org.apache.hadoop.yarn.server.nodemanager.containermanager.monitor.ContainerMetrics.usageMetrics
>  is a mutable collection which should be package protected  At 
> ContainerMetrics.java:[line 134]
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to