Skip to content

Commit

Permalink
logstore: resolve a comment
Browse files Browse the repository at this point in the history
Epic: none
Release note: None
  • Loading branch information
tbg committed Dec 16, 2022
1 parent 883238d commit bf41206
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions pkg/kv/kvserver/logstore/logstore.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,9 +104,8 @@ type LogStore struct {
}

func newStoreEntriesBatch(eng storage.Engine) storage.Batch {
// TODO(pavelkalinnikov): Doesn't this comment contradict the code?
// Use a more efficient write-only batch because we don't need to do any
// reads from the batch. Any reads are performed on the underlying DB.
// Use an unindexed batch because we don't need to read our writes, and
// it is more efficient.
return eng.NewUnindexedBatch(false /* writeOnly */)
}

Expand Down

0 comments on commit bf41206

Please sign in to comment.