diff options
| author | Andrew Kryczka <andrewkr@fb.com> | 2019-01-02 15:05:41 -0800 |
|---|---|---|
| committer | Facebook Github Bot <facebook-github-bot@users.noreply.github.com> | 2019-01-02 15:08:53 -0800 |
| commit | ace543a815125cd82ee50727572ba4c383a5b848 (patch) | |
| tree | a9cba9cce1d44f1454926e5dad15bff6a16d6d1f /HISTORY.md | |
| parent | e8210e44dae8154f0d9f29e15dfa1158d2b84395 (diff) | |
fix accounting for range tombstones in TableProperties (#4841)
Summary:
- To be consistent with the accounting of other optypes in `TableProperties`, we should count range tombstones in `TableProperties::num_entries` and `TableProperties::num_deletions`.
- Updated assertions in stress test's `OnTableFileCreated` handler to accept files with range tombstones only.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/4841
Differential Revision: D13568424
Pulled By: ajkr
fbshipit-source-id: 0139d7806494eda20ece67ec460d2458dbbf6026
Diffstat (limited to 'HISTORY.md')
| -rw-r--r-- | HISTORY.md | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/HISTORY.md b/HISTORY.md index 161b2597a..113381d8c 100644 --- a/HISTORY.md +++ b/HISTORY.md @@ -7,6 +7,7 @@ ### Public API Change * Transaction::GetForUpdate is extended with a do_validate parameter with default value of true. If false it skips validating the snapshot before doing the read. Similarly ::Merge, ::Put, ::Delete, and ::SingleDelete are extended with assume_tracked with default value of false. If true it indicates that call is assumed to be after a ::GetForUpdate. +* `TableProperties::num_entries` and `TableProperties::num_deletions` now also account for number of range tombstones. ### Bug Fixes * Fix a deadlock caused by compaction and file ingestion waiting for each other in the event of write stalls. |