Skip to content

Commit 0e34dcb

Browse files
committed
Revert "HBASE-22610 [BucketCache] Rename hbase.offheapcache.minblocksize"
Reason: Deprecated a wrong parameter. This reverts commit fe450b5.
1 parent fe450b5 commit 0e34dcb

File tree

1 file changed

+3
-21
lines changed

1 file changed

+3
-21
lines changed

hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/BlockCacheFactory.java

Lines changed: 3 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -78,38 +78,20 @@ public final class BlockCacheFactory {
7878
/**
7979
* The target block size used by blockcache instances. Defaults to
8080
* {@link HConstants#DEFAULT_BLOCKSIZE}.
81+
* TODO: this config point is completely wrong, as it's used to determine the
82+
* target block size of BlockCache instances. Rename.
8183
*/
82-
public static final String BLOCKCACHE_BLOCKSIZE_KEY = "hbase.blockcache.minblocksize";
84+
public static final String BLOCKCACHE_BLOCKSIZE_KEY = "hbase.offheapcache.minblocksize";
8385

8486
private static final String EXTERNAL_BLOCKCACHE_KEY = "hbase.blockcache.use.external";
8587
private static final boolean EXTERNAL_BLOCKCACHE_DEFAULT = false;
8688

8789
private static final String EXTERNAL_BLOCKCACHE_CLASS_KEY = "hbase.blockcache.external.class";
8890

89-
/**
90-
* @deprecated use {@link BlockCacheFactory#BLOCKCACHE_BLOCKSIZE_KEY} instead.
91-
*/
92-
@Deprecated
93-
static final String DEPRECATED_BLOCKCACHE_BLOCKSIZE_KEY = "hbase.blockcache.minblocksize";
94-
95-
/**
96-
* The config point hbase.offheapcache.minblocksize is completely wrong, which is replaced by
97-
* {@link BlockCacheFactory#BLOCKCACHE_BLOCKSIZE_KEY}. Keep the old config key here for backward
98-
* compatibility.
99-
*/
100-
static {
101-
Configuration.addDeprecation(DEPRECATED_BLOCKCACHE_BLOCKSIZE_KEY, BLOCKCACHE_BLOCKSIZE_KEY);
102-
}
103-
10491
private BlockCacheFactory() {
10592
}
10693

10794
public static BlockCache createBlockCache(Configuration conf) {
108-
if (conf.get(DEPRECATED_BLOCKCACHE_BLOCKSIZE_KEY) != null) {
109-
LOG.warn("The config key {} is deprecated now, instead please use {}. In future release "
110-
+ "we will remove the deprecated config.", DEPRECATED_BLOCKCACHE_BLOCKSIZE_KEY,
111-
BLOCKCACHE_BLOCKSIZE_KEY);
112-
}
11395
FirstLevelBlockCache l1Cache = createFirstLevelCache(conf);
11496
if (l1Cache == null) {
11597
return null;

0 commit comments

Comments
 (0)