-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: support concurrently insert batches into memtable #81
Comments
The current Skiplist approach doesn't support concurrent inserting. Maybe we should port |
I would like to try on this. |
@accelsao Also you may refer to https://github.com/dgraph-io/badger/blob/master/skl/skl.go, which looks like a more clean version of the rocksdb's |
badger arena is different from rocksbdb, do you know which one is better to write in rust? |
I think they serve the same purpose with a different approach. Using the current arena should be enough. |
How do you think about |
It may need some extra work to fit the |
@accelsao I've tested the offset-based |
Can you help me? I am quite busy now QQ |
In the origin version of leveldb, the write path is like below:
Writing batches concurrently will result in a single thread ingestion into memtable and then notify all the relate blocked write request threads.
Maybe we can support concurrently inserting batches into memtable , which should improve the write performance a lot.
This feature includes following tasks:
The text was updated successfully, but these errors were encountered: