address comments

Signed-off-by: tabokie <xy.tao@outlook.com>
tikv · Jun 28, 2023 · dedc836 · dedc836
1 parent 81175fd
commit dedc836
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/components/engine_rocks/src/misc.rs b/components/engine_rocks/src/misc.rs
@@ -42,8 +42,7 @@ impl RocksEngine {
                 .as_ref()
                 .map_or(false, |key| key.as_slice() > r.start_key)
             {
-                self.delete_all_in_range_cf_by_key(wopts, cf, &r)?;
-                written = true;
+                written |= self.delete_all_in_range_cf_by_key(wopts, cf, &r)?;
                 continue;
             }
             last_end_key = Some(r.end_key.to_owned());

diff --git a/tests/integrations/raftstore/test_single.rs b/tests/integrations/raftstore/test_single.rs
@@ -92,7 +92,7 @@ fn test_delete() {
 }
 
 #[test_case(test_raftstore::new_node_cluster)]
-// v2 doesn't support.
+// v2 doesn't support RocksDB delete range.
 fn test_node_use_delete_range() {
     let mut cluster = new_cluster(0, 1);
     cluster.cfg.raft_store.use_delete_range = true;