Gera Shegalov created HADOOP-11659:
--------------------------------------
Summary: o.a.h.fs.FileSystem.Cache#remove should use a single hash
map lookup
Key: HADOOP-11659
URL: https://issues.apache.org/jira/browse/HADOOP-11659
Project: Hadoop Common
Issue Type: Improvement
Components: fs
Affects Versions: 2.6.0
Reporter: Gera Shegalov
Priority: Trivial
The method looks up the same key in the same hash map potentially 3 times
{code}
if (map.containsKey(key) && fs == map.get(key)) {
map.remove(key)
{code}
Instead it could do a single lookup
{code}
FileSystem cachedFs = map.remove(key);
{code}
and then test cachedFs == fs or something else.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
