-
Notifications
You must be signed in to change notification settings - Fork 87
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
add --nocache option to disable compact block file cache #483
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
The changes here look good to me, and are exactly what I had in mind.
fa286fc
to
668cd0f
Compare
Force pushed review suggestion, thanks @arya2. Perhaps you can review again and approve? I think this is ready to be merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
This fixes issue zcash#480. Enabling this option decreases the performance of gRPCs GetBlock and GetBlockRange, but conserves disk space (the size of the database is currently around 17GB). Note, specifying this option will also delete the existing cache (so if you start again without --nocache, the cache will need to be recreated).
668cd0f
to
e0503c4
Compare
Force pushed to rebase |
Closes #480. Enabling this option decreases the performance of gRPCs GetBlock and GetBlockRange, but conserves disk space (the current database size is about 17GB).
Note, specifying this option will also delete the existing cache (so if you start again without --nocache, the cache will need to be recreated).