Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

[beta] kvdb backports #7438

Merged
merged 2 commits into from
Jan 3, 2018
Merged

[beta] kvdb backports #7438

merged 2 commits into from
Jan 3, 2018

Conversation

andresilva
Copy link
Contributor

debris and others added 2 commits January 3, 2018 13:22
@debris @andresilva
Merge pull request #6720 from paritytech/kvdb_split
61c3e1a 
separated kvdb into 3 crates: kvdb, kvdb-memorydb && kvdb-rocksdb
@andresilva
Upgrade to RocksDB 5.8.8 and tune settings to reduce space amplificat…
e93872e 
…ion (#7348)

* kvdb-rocksdb: update to RocksDB 5.8.8

* kvdb-rocksdb: tune RocksDB options

* Switch to level-style compaction
* Increase default block size (16K), and use bigger blocks for HDDs (64K)
* Increase default file size base (64MB SSDs, 256MB HDDs)
* Create a single block cache shared across all column families
* Tune compaction settings using RocksDB helper functions, taking into account
  memory budget spread across all columns
* Configure backgrounds jobs based on the number of CPUs
* Set some default recommended settings

* ethcore: remove unused config blockchain.db_cache_size

* parity: increase default value for db_cache_size

* kvdb-rocksdb: enable compression on all levels

* kvdb-rocksdb: set global db_write_bufer_size

* kvdb-rocksdb: reduce db_write_bufer_size to force earlier flushing

* kvdb-rocksdb: use master branch for rust-rocksdb dependency
@parity-cla-bot
Copy link

It looks like @andresilva signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@5chdn 5chdn added A0-pleasereview 🤓 Pull request needs code review. A8-backport 🕸 Pull request is already reviewed well in another branch. M4-core ⛓ Core client code / Rust. labels Jan 3, 2018
@5chdn 5chdn added this to the Patch milestone Jan 3, 2018
@5chdn 5chdn added the P2-asap 🌊 No need to stop dead in your tracks, however issue should be addressed as soon as possible. label Jan 3, 2018
@tomusdrw tomusdrw added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jan 3, 2018
@debris debris merged commit 849e574 into beta Jan 3, 2018
@debris debris deleted the beta-backports-kvdb branch January 3, 2018 15:09
@iostat iostat mentioned this pull request Jan 8, 2018
Closed
@lght lght mentioned this pull request Jan 9, 2018
Closed
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tomusdrw tomusdrw tomusdrw approved these changes

Assignees
No one assigned
Labels
A8-backport 🕸 Pull request is already reviewed well in another branch. A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust. P2-asap 🌊 No need to stop dead in your tracks, however issue should be addressed as soon as possible.
Projects
None yet
Milestone
Patch
Development

Successfully merging this pull request may close these issues.
5 participants
@andresilva @parity-cla-bot @tomusdrw @debris @5chdn