Differential D10277

[avalanche] Clear timed out proof poll requests
ClosedPublic
Actions

Authored by Fabien on Oct 7 2021, 12:31.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABC14dde6de82f1: [avalanche] Clear timed out proof poll requests

Summary

As per title, demonstrated by reusing the existing unit test but with proofs.
Documentation for the explicit decorator declaration: https://www.boost.org/doc/libs/1_66_0/libs/test/doc/html/boost_test/tests_organization/decorators/explicit_decorator_declaration.html

Ref T1854.

Depends on D10271.

Test Plan

ninja check-avalanche-processor_tests

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_get_proof_from_proofid

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16922
Build 33681: Build Diff	lint-circular-dependencies · build-diff · build-without-wallet · build-clang-tidy · build-clang · build-debug
Build 33680: arc lint + arc unit

Event Timeline

Fabien created this revision.Oct 7 2021, 12:31

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 7 2021, 12:31

Fabien requested review of this revision.Oct 7 2021, 12:31

Harbormaster completed remote builds in B16922: Diff 30367.Oct 7 2021, 12:49

Fabien added a child revision: D10278: [avalanche] Limit the number of inflight proof poll requests.Oct 7 2021, 13:15

PiRK accepted this revision.Oct 11 2021, 10:34

This revision is now accepted and ready to land.Oct 11 2021, 10:34

Closed by commit rABC14dde6de82f1: [avalanche] Clear timed out proof poll requests (authored by Fabien). · Explain WhyOct 12 2021, 08:52

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC14dde6de82f1: [avalanche] Clear timed out proof poll requests.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

processor.cpp

35 lines

test/

processor_tests.cpp

25 lines

Diff 30367

View Options

src/avalanche/processor.cpp