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

blockpool stability fixes: #702

Merged
merged 1 commit into from
Apr 13, 2015

Conversation

zelig
Copy link
Contributor

@zelig zelig commented Apr 13, 2015

  • follow up locks and fix them
  • chainManager: call SetQueued for parentErr future blocks, uncomment TD checks, unskip test
  • make ErrIncorrectTD non-fatal to be forgiving to genuine mistaken nodes (temp) but demote them to guard against stuck best peers.
  • add purging to bounded nodeCache (config nodeCacheSize)
  • use nodeCache when creating blockpool entries and let non-best peers add blocks (performance boost)
  • minor error in addError
  • reduce idleBestPeerTimeout to 1 minute
  • correct status counts and unskip status passing status test
  • glogified logging

- follow up locks and fix them
- chainManager: call SetQueued for parentErr future blocks, uncomment TD checks, unskip test
- make ErrIncorrectTD non-fatal to be forgiving to genuine mistaken nodes (temp) but demote them to guard against stuck best peers.
- add purging to bounded nodeCache (config nodeCacheSize)
- use nodeCache when creating blockpool entries and let non-best peers add blocks (performance boost)
- minor error in addError
- reduce idleBestPeerTimeout to 1 minute
- correct status counts and unskip status passing status test
- glogified logging
@zelig zelig force-pushed the frontier/blockpool branch from 6bb83a1 to 3d57e37 Compare April 13, 2015 12:14
@zelig zelig self-assigned this Apr 13, 2015
obscuren added a commit that referenced this pull request Apr 13, 2015
@obscuren obscuren merged commit 1fa844a into ethereum:develop Apr 13, 2015
@zelig zelig mentioned this pull request Apr 14, 2015
13 tasks
@zelig zelig deleted the frontier/blockpool branch December 8, 2015 20:51
@zelig zelig restored the frontier/blockpool branch December 8, 2015 20:51
@gbalint gbalint deleted the frontier/blockpool branch May 25, 2018 14:46
tony-ricciardi pushed a commit to tony-ricciardi/go-ethereum that referenced this pull request Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants