Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Check queue to determine major importing #2763

Merged
merged 14 commits into from
Oct 20, 2016
Merged

Check queue to determine major importing #2763

merged 14 commits into from
Oct 20, 2016

Conversation

keorn
Copy link

@keorn keorn commented Oct 20, 2016 •
edited
Loading

Take into account both the SyncState and queue state to determine if major_importing is going on (many blocks imported at a time). Works with parity import now.
Location in rpc suggested by @arkpar .

  • do some live testing

@keorn keorn added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M4-core ⛓ Core client code / Rust. A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Oct 20, 2016
@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 20, 2016
@gavofyork gavofyork merged commit 866ab9c into master Oct 20, 2016
@gavofyork gavofyork deleted the eth-syncing branch October 20, 2016 21:36
@keorn keorn mentioned this pull request Oct 20, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@keorn @gavofyork @arkpar