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

Fix the dumpblockchain function #405

Merged
merged 1 commit into from
Oct 10, 2016

Conversation

cjepson
Copy link
Contributor

@cjepson cjepson commented Oct 6, 2016

The dumpblockchain function used to serialize a map of block
into gob serialized format, which was used for testing but which
was incompatible with the addblock tool. The function now dumps
a flat file the the same format required by the addblock tool.

A couple shutdown assertions were added as well, to prevent
potential panics if pointers were nil. The duration of time
it took to sync the blockchain with addblock is now
reported.

@jolan
Copy link
Contributor

jolan commented Oct 7, 2016

Can you please change the description in config.go to match? Maybe something like "Write blockchain in addblock format to the specified file"

The dumpblockchain function used to serialize a map of block
into gob serialized format, which was used for testing but which
was incompatible with the addblock tool.  The function now dumps
a flat file the the same format required by the addblock tool.

A couple shutdown assertions were added as well, to prevent
potential panics if pointers were nil.  The duration of time
it took to sync the blockchain with addblock is now
reported.
@cjepson
Copy link
Contributor Author

cjepson commented Oct 7, 2016

@jolan Fixed

Copy link
Contributor

@jolan jolan left a comment

Choose a reason for hiding this comment

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

Tested and works as expected.

OK

@alexlyp alexlyp merged commit 393c48d into decred:master Oct 10, 2016
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