From e0906eb785d27d963edd5d3330dafd0c3c795179 Mon Sep 17 00:00:00 2001
From: Maysam Yabandeh
Date: Fri, 27 Jul 2018 09:26:26 -0700
Subject: [PATCH] Clarify max_total_wal_size's scope (#4194)
Summary:
max_total_wal_size takes effect only when there are more than one column families. The patch clarify that in the inline docs
Closes https://github.com/facebook/rocksdb/issues/4180
Pull Request resolved: https://github.com/facebook/rocksdb/pull/4194
Differential Revision: D9028767
Pulled By: maysamyabandeh
fbshipit-source-id: 8d730ca7f15e76e7ee9ff88b2b48030b2d1b7078
---
include/rocksdb/options.h | 2 ++
java/src/main/java/org/rocksdb/DBOptionsInterface.java | 2 ++
2 files changed, 4 insertions(+)
diff --git a/include/rocksdb/options.h b/include/rocksdb/options.h
index 4a525365c..f2966ef79 100644
--- a/include/rocksdb/options.h
+++ b/include/rocksdb/options.h
@@ -430,6 +430,8 @@ struct DBOptions {
// (i.e. the ones that are causing all the space amplification). If set to 0
// (default), we will dynamically choose the WAL size limit to be
// [sum of all write_buffer_size * max_write_buffer_number] * 4
+ // This option takes effect only when there are more than one column family as
+ // otherwise the wal size is dictated by the write_buffer_size.
// Default: 0
uint64_t max_total_wal_size = 0;
diff --git a/java/src/main/java/org/rocksdb/DBOptionsInterface.java b/java/src/main/java/org/rocksdb/DBOptionsInterface.java
index f14a80e4e..7c406eaf8 100644
--- a/java/src/main/java/org/rocksdb/DBOptionsInterface.java
+++ b/java/src/main/java/org/rocksdb/DBOptionsInterface.java
@@ -262,6 +262,8 @@ public interface DBOptionsInterface {
*
* If set to 0 (default), we will dynamically choose the WAL size limit to
* be [sum of all write_buffer_size * max_write_buffer_number] * 2
+ * This option takes effect only when there are more than one column family as
+ * otherwise the wal size is dictated by the write_buffer_size.
* Default: 0
*
* @param maxTotalWalSize max total wal size.