[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-2732?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rakesh R resolved ZOOKEEPER-2732.
---------------------------------
    Resolution: Fixed

Issue resolved by pull request 258
[https://github.com/apache/zookeeper/pull/258]

> Cleanup findbug warnings in branch-3.4: Performance Warnings
> ------------------------------------------------------------
>
>                 Key: ZOOKEEPER-2732
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2732
>             Project: ZooKeeper
>          Issue Type: Sub-task
>            Reporter: Rakesh R
>            Assignee: Abraham Fine
>             Fix For: 3.4.11
>
>
> Please refer the attached sheet in parent jira. Below is the details of 
> findbug warnings.
> {code}
> Bx    Boxing/unboxing to parse a primitive new 
> org.apache.zookeeper.server.quorum.QuorumCnxManager(long, Map, 
> QuorumAuthServer, QuorumAuthLearner, int, boolean, int, boolean)
> Bx    new org.apache.zookeeper.server.quorum.QuorumCnxManager(long, Map, 
> QuorumAuthServer, QuorumAuthLearner, int, boolean, int, boolean) invokes 
> inefficient new Integer(String) constructor; use Integer.valueOf(String) 
> instead
> Dm    
> org.apache.zookeeper.server.quorum.FastLeaderElection$Notification.toString() 
> invokes inefficient new String(String) constructor
> WMI   org.apache.zookeeper.server.DataTree.dumpEphemerals(PrintWriter) makes 
> inefficient use of keySet iterator instead of entrySet iterator
> WMI   
> org.apache.zookeeper.server.quorum.flexible.QuorumHierarchical.computeGroupWeight()
>  makes inefficient use of keySet iterator instead of entrySet iterator
> WMI   
> org.apache.zookeeper.server.quorum.flexible.QuorumHierarchical.containsQuorum(HashSet)
>  makes inefficient use of keySet iterator instead of entrySet iterator
> WMI   org.apache.zookeeper.ZooKeeperMain.usage() makes inefficient use of 
> keySet iterator instead of entrySet iterator
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to