big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

HDFS-15907. Reduce Memory Overhead of AclFeature by avoiding AtomicInteger. Contributed by Stephen O'Donnell.

Browse Source
This commit is contained in:
Ayush Saxena 2021-03-24 02:10:02 +05:30
parent 78bddd0d9f
commit 76c40a52d7
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/AclFeature.java
View File

@ -19,7 +19,6 @@
package org.apache.hadoop.hdfs.server.namenode;
import java.util.Arrays;
import java.util.concurrent.atomic.AtomicInteger;
import org.apache.hadoop.classification.InterfaceAudience;
import org.apache.hadoop.fs.permission.AclEntry;
@ -35,7 +34,7 @@
public class AclFeature implements INode.Feature, ReferenceCounter {
public static final ImmutableList<AclEntry> EMPTY_ENTRY_LIST =
ImmutableList.of();
private AtomicInteger value = new AtomicInteger();
private int refCount = 0;
private final int [] entries;
@ -84,17 +83,17 @@ public int hashCode() {
}
@Override
public int getRefCount() {
return value.get();
public synchronized int getRefCount() {
return refCount;
}
@Override
public int incrementAndGetRefCount() {
return value.incrementAndGet();
public synchronized int incrementAndGetRefCount() {
return ++refCount;
}
@Override
public int decrementAndGetRefCount() {
return value.updateAndGet(i -> i > 0 ? i - 1 : i);
public synchronized int decrementAndGetRefCount() {
return (refCount > 0) ? --refCount : 0;
}
}