BUG FIX: set target file size multiplier

[yehaolan]

Overview

documentation of targetFileSizeBase and targetFileSizeMultiplier is located here (texted copied below)

targetFileSizeBase determines a level-1 file size. Target file size for level L can be calculated by targetFileSizeBase * (targetFileSizeMultiplier ^ (L-1)) For example, if targetFileSizeBase is 2MB and target_file_size_multiplier is 10, then each file on level-1 will be 2MB, and each file on level 2 will be 20MB, and each file on level-3 will be 200MB. by default targetFileSizeBase is 64MB.

Before the change

samza/samza-kv-rocksdb/src/main/java/org/apache/samza/storage/kv/RocksDbOptionsHelper.java

Lines 178 to 184 in f9c3241

	if (storeConfig.containsKey(ROCKSDB_COMPACTION_TARGET_FILE_SIZE_BASE)) {
	options.setTargetFileSizeBase(storeConfig.getLong(ROCKSDB_COMPACTION_TARGET_FILE_SIZE_BASE));
	}
	if (storeConfig.containsKey(ROCKSDB_COMPACTION_TARGET_FILE_SIZE_MULTIPLIER)) {
	options.setTargetFileSizeBase(storeConfig.getLong(ROCKSDB_COMPACTION_TARGET_FILE_SIZE_MULTIPLIER));
	}

Behavior:

• If ROCKSDB_COMPACTION_TARGET_FILE_SIZE_BASE is configured, the value of ROCKSDB_COMPACTION_TARGET_FILE_SIZE_BASE will be set as the target file size base.
• If ROCKSDB_COMPACTION_TARGET_FILE_SIZE_MULTIPLIER is configured, the value of ROCKSDB_COMPACTION_TARGET_FILE_SIZE_MULTIPLIER will be set as the target file size base (it's wrong, and the file size multiplier should be set instead).

After the change

Behavior:

• If ROCKSDB_COMPACTION_TARGET_FILE_SIZE_BASE is configured, the value of ROCKSDB_COMPACTION_TARGET_FILE_SIZE_BASE will be set as the target file size base.
• If ROCKSDB_COMPACTION_TARGET_FILE_SIZE_MULTIPLIER is configured, the value of ROCKSDB_COMPACTION_TARGET_FILE_SIZE_MULTIPLIER will be set as the file size multiplier.

---

Tests

• Run ./gradlew build

[ajothomas]

Thanks for the fix.