Page MenuHomePhabricator

[avalanche] Limit the number of in flight request per item.
ClosedPublic

Authored by deadalnix on Feb 3 2019, 16:43.

Details

Summary

It is paramount that avalanche is able to pace itself based on the speed at which the peer can respond.

Depends on D2474, D2473, D2472, D2470, D2484, D2485 and D2471

Test Plan

Added test cases.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avainflight
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4847
Build 7757: Bitcoin ABC Buildbot (legacy)
Build 7756: arc lint + arc unit

Event Timeline

jasonbcox added a subscriber: jasonbcox.

Some nits. All else LGTM.

src/avalanche.cpp
45

clearInflighRequest -> clearInflightRequest

390

could use auto here instead of writing out std::pair<...>

src/test/avalanche_tests.cpp
119

everythign -> everything

This revision is now accepted and ready to land.Feb 3 2019, 20:05
This revision was automatically updated to reflect the committed changes.