Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-22447 Check refCount before free block in BucketCache

This commit is contained in:
binlijin 2019-05-22 16:19:08 +08:00
parent a30b186568
commit 2bf7ad4e4f
1 changed files with 10 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/bucket/BucketCache.java
View File

@ -557,37 +557,6 @@ public class BucketCache implements BlockCache, HeapSize {
return evictBlock(cacheKey, true); return evictBlock(cacheKey, true);
} }
// does not check for the ref count. Just tries to evict it if found in the
// bucket map
private boolean forceEvict(BlockCacheKey cacheKey) {
if (!cacheEnabled) {
return false;
}
RAMQueueEntry removedBlock = checkRamCache(cacheKey);
BucketEntry bucketEntry = backingMap.get(cacheKey);
if (bucketEntry == null) {
if (removedBlock != null) {
cacheStats.evicted(0, cacheKey.isPrimary());
return true;
} else {
return false;
}
}
ReentrantReadWriteLock lock = offsetLock.getLock(bucketEntry.offset());
try {
lock.writeLock().lock();
if (backingMap.remove(cacheKey, bucketEntry)) {
blockEvicted(cacheKey, bucketEntry, removedBlock == null);
} else {
return false;
}
} finally {
lock.writeLock().unlock();
}
cacheStats.evicted(bucketEntry.getCachedTime(), cacheKey.isPrimary());
return true;
}
private RAMQueueEntry checkRamCache(BlockCacheKey cacheKey) { private RAMQueueEntry checkRamCache(BlockCacheKey cacheKey) {
RAMQueueEntry removedBlock = ramCache.remove(cacheKey); RAMQueueEntry removedBlock = ramCache.remove(cacheKey);
if (removedBlock != null) { if (removedBlock != null) {
@ -1049,8 +1018,15 @@ public class BucketCache implements BlockCache, HeapSize {
ReentrantReadWriteLock lock = offsetLock.getLock(bucketEntries[i].offset()); ReentrantReadWriteLock lock = offsetLock.getLock(bucketEntries[i].offset());
try { try {
lock.writeLock().lock(); lock.writeLock().lock();
if (backingMap.remove(key, bucketEntries[i])) { int refCount = bucketEntries[i].getRefCount();
blockEvicted(key, bucketEntries[i], false); if (refCount == 0) {
if (backingMap.remove(key, bucketEntries[i])) {
blockEvicted(key, bucketEntries[i], false);
} else {
bucketEntries[i].markForEvict();
}
} else {
bucketEntries[i].markForEvict();
} }
} finally { } finally {
lock.writeLock().unlock(); lock.writeLock().unlock();
@ -1693,7 +1669,7 @@ public class BucketCache implements BlockCache, HeapSize {
if (bucketEntry != null) { if (bucketEntry != null) {
int refCount = bucketEntry.decrementRefCountAndGet(); int refCount = bucketEntry.decrementRefCountAndGet();
if (refCount == 0 && bucketEntry.isMarkedForEvict()) { if (refCount == 0 && bucketEntry.isMarkedForEvict()) {
forceEvict(cacheKey); evictBlock(cacheKey);
} }
} }
} }