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

[Beta] Backports #7756

Merged
merged 3 commits into from
Jan 31, 2018
Merged

[Beta] Backports #7756

merged 3 commits into from
Jan 31, 2018

Conversation

andresilva
Copy link
Contributor

@andresilva andresilva commented Jan 31, 2018

* Filter-out nodes.json

* network: sort node table nodes by failure ratio

* network: fix node table tests

* network: fit node failure percentage into buckets of 5%

* network: consider number of attempts in sorting of node table

* network: fix node table grumbles
@parity-cla-bot
Copy link

It looks like @andresilva signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@andresilva andresilva changed the base branch from master to beta January 31, 2018 11:22
@andresilva andresilva added A0-pleasereview 🤓 Pull request needs code review. A8-backport 🕸 Pull request is already reviewed well in another branch. M4-core ⛓ Core client code / Rust. labels Jan 31, 2018
@5chdn 5chdn added this to the Patch milestone Jan 31, 2018
andresilva and others added 2 commits January 31, 2018 12:08
* parity: wait for client to drop on shutdown

* parity: fix grumbles in shutdown wait

* parity: increase shutdown timeouts
* Update Clap dependency and remove workarounds

* WIP

* Remove line breaks in help messages for now

* Multiple values can only be separated by commas (closes #7428)

* Grumbles; refactor repeating code; add constant

* Use a single Wrapper rather than allocate a new one for each call

* Wrap --help to 120 characters rather than 100 characters
@5chdn 5chdn merged commit a42d780 into beta Jan 31, 2018
@5chdn 5chdn deleted the beta-backports branch January 31, 2018 20:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review. A8-backport 🕸 Pull request is already reviewed well in another branch. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants