Page MenuHomePhabricator
Differential D2121

[avalanche] Add support to poll multiple block at once.
ClosedPublic
Actions

Authored by deadalnix on Nov 22 2018, 18:44.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rSTAGING81fcace04fdf: [avalanche] Add support to poll multiple block at once.
rABC81fcace04fdf: [avalanche] Add support to poll multiple block at once.
Summary

Add support and test to poll multiple blocks at once. Block are selected making sure the block with most PoW are selected first.

Depends on D2044

Test Plan

Added test cases.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

deadalnix created this revision.Nov 22 2018, 18:44
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 22 2018, 18:44
deadalnix added a child revision: D2004: Reconsider parked chains when enough works piles up.Nov 22 2018, 18:49
deadalnix removed a child revision: D2004: Reconsider parked chains when enough works piles up.
deadalnix added a child revision: D2045: [avalanche] Report the list of finalized blocks when votes are accounted for.
deadalnix updated this revision to Diff 6036.Nov 22 2018, 19:48

rebase

teamcity added a subscriber: teamcity.Nov 22 2018, 20:33

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

teamcity added a comment.Nov 22 2018, 20:37

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

teamcity added a comment.Nov 22 2018, 20:41

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4064: Diff 6032.Nov 22 2018, 20:41
teamcity added a comment.Nov 22 2018, 20:45

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4067: Diff 6036.Nov 22 2018, 20:45
Fabien accepted this revision.Nov 22 2018, 23:00
This revision is now accepted and ready to land.Nov 22 2018, 23:00
Fabien added inline comments.Nov 22 2018, 23:02
src/test/avalanche_tests.cpp
191 ↗(On Diff #6036)

voing => voting

Closed by commit rABC81fcace04fdf: [avalanche] Add support to poll multiple block at once. (authored by deadalnix). · Explain WhyNov 22 2018, 23:09
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
15 lines
test/
61 lines

Diff 6045

View Options

src/avalanche.cpp

Loading...
View Options

src/test/avalanche_tests.cpp

Loading...