0a43d8a261
Summary: Continuing cbi42 's work in 602cc0f9a4be89020fb870dba2816f11dd515d16. In this PR, we are adding record count verification for each compaction by comparing number of entries summed from Table Properties with the number of output records from the compaction stats. If the count does not match, `Status::Corruption(msg)` is returned with detailed message including the actual number (from table property) and the expected number (from compaction stats) Pull Request resolved: https://github.com/facebook/rocksdb/pull/13455 Test Plan: New UT added ``` ./db_compaction_test -- --gtest_filter="*Verify*" ``` The check had to be disabled for some of the existing tests using MockTable/MockTableFactory, because TableProperties aren't populated properly for the MockTables. Reviewed By: hx235 Differential Revision: D71235790 Pulled By: jaykorean fbshipit-source-id: 3a86a878d13e79d948409d6a9843d1c992d2c98e |
||
|---|---|---|
| .. | ||
| plain_table_bloom.cc | ||
| plain_table_bloom.h | ||
| plain_table_builder.cc | ||
| plain_table_builder.h | ||
| plain_table_factory.cc | ||
| plain_table_factory.h | ||
| plain_table_index.cc | ||
| plain_table_index.h | ||
| plain_table_key_coding.cc | ||
| plain_table_key_coding.h | ||
| plain_table_reader.cc | ||
| plain_table_reader.h | ||