You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Hiroaki Yoshida (JIRA)" <ji...@apache.org> on 2018/08/21 11:20:00 UTC

[jira] [Created] (FLINK-10189) FindBugs warnings: Inefficient use of keySet iterator instead of entrySet iterator

Hiroaki Yoshida created FLINK-10189:
---------------------------------------

             Summary: FindBugs warnings: Inefficient use of keySet iterator instead of entrySet iterator
                 Key: FLINK-10189
                 URL: https://issues.apache.org/jira/browse/FLINK-10189
             Project: Flink
          Issue Type: Bug
            Reporter: Hiroaki Yoshida


FindBugs-3.0.1 ([http://findbugs.sourceforge.net/]) reported two WMI_WRONG_MAP_ITERATOR warnings on master:
{code:java}
M P WMI: org.apache.flink.runtime.state.ttl.TtlMapState.putAll(Map) makes inefficient use of keySet iterator instead of entrySet iterator  At TtlMapState.java:[line 72]
M P WMI: org.apache.flink.addons.hbase.HBaseTableSource.projectFields(int[]) makes inefficient use of keySet iterator instead of entrySet iterator  At HBaseTableSource.java:[line 19] 
{code}
The description of the bug is as follows:
{quote}*WMI: Inefficient use of keySet iterator instead of entrySet iterator (WMI_WRONG_MAP_ITERATOR)*
This method accesses the value of a Map entry, using a key that was retrieved from a keySet iterator. It is more efficient to use an iterator on the entrySet of the map, to avoid the Map.get(key) lookup.
[http://findbugs.sourceforge.net/bugDescriptions.html#WMI_WRONG_MAP_ITERATOR]
{quote}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)