Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Sep 17, 2018

When sending transactions to peers, we announce them to different "buckets" at different times. Currently there is a bucket for each outgoing peer and one bucket for all incoming peers (see #13298)

This tests checks that we don't send transactions to peers in one bucket when they were never announced in that bucket. For simplicity, the test only considers one outbound peer and one inbound peer (two buckets).

@DrahtBot
Copy link
Contributor

DrahtBot commented Sep 24, 2018

Note to reviewers: This pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@maflcko maflcko closed this Sep 26, 2018
@maflcko maflcko deleted the Mf1809-qaBucket branch September 26, 2018 19:31
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants