Skip to content

options: add set_verify_checksums and set_level_compaction_dynamic_level_bytes#349

Merged
aleksuss merged 9 commits into
rust-rocksdb:masterfrom
ordian:master
Nov 12, 2019
Merged

options: add set_verify_checksums and set_level_compaction_dynamic_level_bytes#349
aleksuss merged 9 commits into
rust-rocksdb:masterfrom
ordian:master

Conversation

@ordian

@ordian ordian commented Nov 9, 2019

Copy link
Copy Markdown

Part of #260.

Comment thread src/db.rs
@aleksuss

aleksuss commented Nov 9, 2019

Copy link
Copy Markdown
Member

It would be cool if you'll add unit tests for these methods.

@ordian

ordian commented Nov 9, 2019

Copy link
Copy Markdown
Author

@aleksuss please take another look

@aleksuss

aleksuss commented Nov 9, 2019

Copy link
Copy Markdown
Member

One more thing. cargo fmt apply please.

@aleksuss aleksuss merged commit 6c7f6af into rust-rocksdb:master Nov 12, 2019
rleungx pushed a commit to rleungx/rust-rocksdb that referenced this pull request Jun 17, 2020
Add OnBackgroundError handler API. 

For the API, also adding crocksdb_status_ptr_t struct, which may be later extend to allow setting a different status (since rocksdb OnBackgroundError allow overriding the background error).

Signed-off-by: Yi Wu <[email protected]>
BusyJay pushed a commit to BusyJay/rust-rocksdb that referenced this pull request Jul 23, 2020
Add OnBackgroundError handler API.

For the API, also adding crocksdb_status_ptr_t struct, which may be later extend to allow setting a different status (since rocksdb OnBackgroundError allow overriding the background error).

Signed-off-by: Yi Wu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants