forked from apache/bookkeeper
-
Notifications
You must be signed in to change notification settings - Fork 4
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
bring back deleteRange for RocksDB to improve location delete perform…
…ance (apache#3653) The entry log location index deletion is deleted in batches one by one currently, and it will have low performance. Refer to: apache#3646 Matteo has introduced deleteRange API a few years ago, but rollback due to RocksDB delete ranges bug. apache#1620. The RocksDB bug has been addressed since 5.18.0 https://github.com/facebook/rocksdb/blob/main/HISTORY.md#5180-2018-11-30. We can bring the `deleteRange` API back to improve the entry log location deletion performance. Bring `deleteRange` API back for entry log location deletion. (cherry picked from commit 696919c) (cherry picked from commit e159510)
- Loading branch information
1 parent
7d80a31
commit 1ed2463
Showing
2 changed files
with
73 additions
and
60 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters