From 1338f1b09942591dc9a68f1dbdd29670395e8806 Mon Sep 17 00:00:00 2001 From: Elephas <72189234+elephas00@users.noreply.github.com> Date: Fri, 15 Mar 2024 10:04:49 +0800 Subject: [PATCH] fix(to issue#455): fix a typo (#456) I am recently debugging at 3B. And when I am reading the Destroy function at file kv/raftstore/peer.go, I found that there is a comment "write kv rocksdb first in case of restart happen between two write". The previous lab in tinykv seems never mention about rocksdb, while the TiKV mentioned. So I think there is a typo, and this comment should be modified to "write kv badgerDB first in case of restart happen between two write". --- kv/raftstore/peer.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kv/raftstore/peer.go b/kv/raftstore/peer.go index c75fe59cc..8d81e0f3e 100644 --- a/kv/raftstore/peer.go +++ b/kv/raftstore/peer.go @@ -209,7 +209,7 @@ func (p *peer) Destroy(engine *engine_util.Engines, keepData bool) error { return err } meta.WriteRegionState(kvWB, region, rspb.PeerState_Tombstone) - // write kv rocksdb first in case of restart happen between two write + // write kv badgerDB first in case of restart happen between two write if err := kvWB.WriteToDB(engine.Kv); err != nil { return err }