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

Configurable rocksdb cache size #1348

Merged
merged 3 commits into from
Jun 20, 2016
Merged

Configurable rocksdb cache size #1348

merged 3 commits into from
Jun 20, 2016

Conversation

NikVolf
Copy link
Contributor

@NikVolf NikVolf commented Jun 20, 2016

No description provided.

@NikVolf NikVolf added the A0-pleasereview 🤓 Pull request needs code review. label Jun 20, 2016
@@ -155,6 +155,7 @@ Footprint Options:
--cache MEGABYTES Set total amount of discretionary memory to use for
the entire system, overrides other cache and queue
options.
--db-cache-size MEGABYTE Database cache size.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps this should be a single --cache options with 20% going to blockchain cache and 80% to db cache, actual ratio needs research though

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ratio could also be configurable in that case.

Copy link
Contributor Author

@NikVolf NikVolf Jun 20, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably later if ratio is not known
database cache was never included in this parameter while being the rocksdb default

@arkpar arkpar added A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jun 20, 2016
@NikVolf NikVolf added A0-pleasereview 🤓 Pull request needs code review. and removed A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. labels Jun 20, 2016
@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jun 20, 2016
@NikVolf NikVolf added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. and removed A8-looksgood 🦄 Pull request is reviewed well. labels Jun 20, 2016
@NikVolf
Copy link
Contributor Author

NikVolf commented Jun 20, 2016

for some reason travis does not run

@rphmeier
Copy link
Contributor

Seems to be in the queue: https://travis-ci.org/ethcore/parity/builds/138874746

@NikVolf NikVolf added A8-looksgood 🦄 Pull request is reviewed well. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Jun 20, 2016
@arkpar arkpar merged commit 8b845e5 into master Jun 20, 2016
@debris debris deleted the db-cache-size branch June 20, 2016 15:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants