Skip to content

Dev Docs: Add Section About Headers-First IBD - #736

Merged
harding merged 2 commits into
bitcoin-dot-org:masterfrom
harding:headers-first
Feb 20, 2015
Merged

Dev Docs: Add Section About Headers-First IBD#736
harding merged 2 commits into
bitcoin-dot-org:masterfrom
harding:headers-first

Conversation

@harding

@harding harding commented Feb 5, 2015

Copy link
Copy Markdown
Contributor

Preview: http://dg0.dtrt.org/en/developer-guide#headers-first

This adds a short section describing headers-first Initial Block Download (IBD) to the developer guide. This was originally part of pull #660, but it has been substansially rewritten. (And, I hope, improved.) Specifics:

  • Add flowchart illustration to blocks-first section to match new headers-first flowchart
  • Add new headers-first (IBD) section
  • Revise Block Broadcasting section to describe non-IBD block transmission workflow
  • Add paragraph about how headers-first avoids handling orphan blocks to the Orphan Blocks section

* Add flowchart illustration to blocks-first section to match new
  headers-first flowchart

* Add new headers-first initial block download (IBD) section

* Revise Block Broadcasting section

* Add paragraph about orphan block handling in headers-first to the
  Orphan Blocks section
Comment thread _includes/guide_p2p_network.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@harding s/a initial/an initial/ maybe?

@ghost1542

Copy link
Copy Markdown
Contributor

@harding Very clear, thanks!

@harding

harding commented Feb 19, 2015

Copy link
Copy Markdown
Contributor Author

@saivann thanks! I fixed the typo you found.

In the absence of critical feedback, this pull will be merged around 13:00 UTC Friday.

@harding
harding merged commit 3d7ba83 into bitcoin-dot-org:master Feb 20, 2015
@harding
harding deleted the headers-first branch February 25, 2015 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants