From 0ea57115a39d8f065444f7b8593feac8672678b1 Mon Sep 17 00:00:00 2001 From: Andrew Kryczka Date: Thu, 31 Jan 2019 14:27:21 -0800 Subject: [PATCH] Fix `WriteBatchBase::DeleteRange` API comment (#4935) Summary: The `DeleteRange` end key is exclusive, not inclusive. Updated API comment accordingly. Pull Request resolved: https://github.com/facebook/rocksdb/pull/4935 Differential Revision: D13905406 Pulled By: ajkr fbshipit-source-id: f577db841a279427991ecf9005cd56b30c8eb3c7 --- include/rocksdb/write_batch_base.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/rocksdb/write_batch_base.h b/include/rocksdb/write_batch_base.h index f91332ee2..a7747a7c8 100644 --- a/include/rocksdb/write_batch_base.h +++ b/include/rocksdb/write_batch_base.h @@ -69,7 +69,7 @@ class WriteBatchBase { const SliceParts& key); virtual Status SingleDelete(const SliceParts& key); - // If the database contains mappings in the range ["begin_key", "end_key"], + // If the database contains mappings in the range ["begin_key", "end_key"), // erase them. Else do nothing. virtual Status DeleteRange(ColumnFamilyHandle* column_family, const Slice& begin_key, const Slice& end_key) = 0;