Skip to content

Commit bdb6dd6

Browse files
author
Sourav Maji
committed
[server] Validate block-cache config against system memory
1 parent 0877e41 commit bdb6dd6

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

Diff for: clients/da-vinci-client/src/test/java/com/linkedin/davinci/store/rocksdb/ReplicationMeadataRocksDBStoragePartitionCFTest.java

+4
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.linkedin.davinci.store.rocksdb;
22

3+
import static com.linkedin.davinci.store.rocksdb.RocksDBServerConfig.ROCKSDB_BLOCK_CACHE_SIZE_IN_BYTES;
4+
import static com.linkedin.davinci.store.rocksdb.RocksDBServerConfig.ROCKSDB_RMD_BLOCK_CACHE_SIZE_IN_BYTES;
35
import static com.linkedin.davinci.store.rocksdb.RocksDBServerConfig.ROCKSDB_SEPARATE_RMD_CACHE_ENABLED;
46
import static org.mockito.ArgumentMatchers.anyInt;
57
import static org.mockito.ArgumentMatchers.anyString;
@@ -49,6 +51,8 @@ public void createStorageEngineForTest() {
4951

5052
Properties properties = new Properties();
5153
properties.put(ROCKSDB_SEPARATE_RMD_CACHE_ENABLED, "true");
54+
properties.put(ROCKSDB_BLOCK_CACHE_SIZE_IN_BYTES, 1024 * 1024 * 1024L);
55+
properties.put(ROCKSDB_RMD_BLOCK_CACHE_SIZE_IN_BYTES, 1024 * 1024L);
5256
VeniceProperties serverProps = AbstractStorageEngineTest.getServerProperties(PersistenceType.ROCKS_DB, properties);
5357
storageService = new StorageService(
5458
AbstractStorageEngineTest.getVeniceConfigLoader(serverProps),

0 commit comments

Comments
 (0)