facebook / facebook/rocksdb

use `OutputValidator` in `SstFileWriter`

Open
#7,599 7 comments 0 reactions 1 assignee Claimed by @briantkim93 View on GitHub
enhancement
Dominant language
C++
Stars
32.1k
Forks
6.9k
Avg merge
32m
Merged PRs (30d)
1

Description

#7134 and #7467 add kv checksum and key order verification to files as they're created via `OutputValidator`. The verifications currently apply to files generated by flush and compaction. It'd be good to introduce these checks for `SstFileWriter`-generated files as well.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.