Skip to content
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

blockchain: Optimize checkpoint handling. #1230

Merged
merged 1 commit into from
Jun 1, 2018

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented May 27, 2018

This requires PR #1229.

This modifies the code that determines the most recently known checkpoint to take advantage of recent changes which make the entire block index available in memory by only storing a reference to the specific node in the index that represents the latest known checkpoint.

Previously, the entire block was stored and new checkpoints required loading it from the database.

This is work towards #1145.

@davecgh davecgh mentioned this pull request May 27, 2018
33 tasks
@davecgh davecgh added this to the 1.3.0 milestone May 27, 2018
@davecgh davecgh force-pushed the blockchain_findcheckpoint_optimize branch from 132fbc9 to 3740a12 Compare May 28, 2018 19:55
@davecgh davecgh force-pushed the blockchain_findcheckpoint_optimize branch from 3740a12 to 5cf5018 Compare June 1, 2018 01:01
This modifies the code that determines the most recently known
checkpoint to take advantage of recent changes which make the entire
block index available in memory by only storing a reference to the
specific node in the index that represents the latest known checkpoint.

Previously, the entire block was stored and new checkpoints required
loading it from the database.
@davecgh davecgh force-pushed the blockchain_findcheckpoint_optimize branch from 5cf5018 to bc96019 Compare June 1, 2018 17:57
@davecgh davecgh merged commit bc96019 into decred:master Jun 1, 2018
@davecgh davecgh deleted the blockchain_findcheckpoint_optimize branch June 1, 2018 18:12
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