HADOOP-11659. o.a.h.FileSystem.Cache#remove should use a single hash map lookup. Contributed by Brahma Reddy Battula.

This commit is contained in:
Akira Ajisaka 2015-03-18 15:43:10 +09:00
parent 5b322c6a82
commit 34117325b2
2 changed files with 10 additions and 4 deletions

View File

@ -449,6 +449,9 @@ Release 2.8.0 - UNRELEASED
HADOOP-11692. Improve authentication failure WARN message to avoid user
confusion. (Yongjun Zhang)
HADOOP-11659. o.a.h.fs.FileSystem.Cache#remove should use a single hash map
lookup. (Brahma Reddy Battula via aajisaka)
OPTIMIZATIONS
BUG FIXES

View File

@ -2700,9 +2700,11 @@ private FileSystem getInternal(URI uri, Configuration conf, Key key) throws IOEx
}
synchronized void remove(Key key, FileSystem fs) {
if (map.containsKey(key) && fs == map.get(key)) {
map.remove(key);
FileSystem cachedFs = map.remove(key);
if (fs == cachedFs) {
toAutoClose.remove(key);
} else if (cachedFs != null) {
map.put(key, cachedFs);
}
}
@ -2730,7 +2732,8 @@ synchronized void closeAll(boolean onlyAutomatic) throws IOException {
}
//remove from cache
remove(key, fs);
map.remove(key);
toAutoClose.remove(key);
if (fs != null) {
try {