From 904aebe7defabdd4c0cfb7076ab8261f129b5ef9 Mon Sep 17 00:00:00 2001 From: Vladimir Petrzhikovskii Date: Wed, 3 Dec 2025 13:05:56 +0100 Subject: [PATCH] Clarify `rocksdb.is-file-deletions-enabled` doc comment --- include/rocksdb/db.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include/rocksdb/db.h b/include/rocksdb/db.h index 2ac8aa99c543..83996a5101f1 100644 --- a/include/rocksdb/db.h +++ b/include/rocksdb/db.h @@ -1291,10 +1291,10 @@ class DB { // filter and index blocks). static const std::string kEstimateTableReadersMem; - // "rocksdb.is-file-deletions-enabled" - returns 0 if deletion of obsolete - // files is enabled; otherwise, returns a non-zero number. - // This name may be misleading because true(non-zero) means disable, - // but we keep the name for backward compatibility. + // "rocksdb.is-file-deletions-enabled" - returns 1 when RocksDB is allowed + // to delete obsolete files (DisableFileDeletions counter is 0), and 0 + // while file deletions are disabled; name kept for backward + // compatibility. static const std::string kIsFileDeletionsEnabled; // "rocksdb.num-snapshots" - returns number of unreleased snapshots of the