Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

p2p perf send a notice of large blocks/txns #408

Closed
pmesnier opened this issue Sep 12, 2017 · 2 comments
Closed

p2p perf send a notice of large blocks/txns #408

pmesnier opened this issue Sep 12, 2017 · 2 comments
Assignees
Milestone

Comments

@pmesnier
Copy link
Contributor

Relates to #291. There is already a notice message, but it isn't being used in the manner described in #291. What I have is a notice message that is only used for transactions and then all recipients immediately turn around and request the contents right away. I'm not entirely sure of the benefit with blocks, since they have to be shared anyway

@pmesnier
Copy link
Contributor Author

pmesnier commented Sep 13, 2017

Marking for M2. There is more discussion required to decide if this issue needs more work or is satisfied by the current code.

@pmesnier
Copy link
Contributor Author

I think this issue is settled. There is a "just send it" max payload size value of 3 times my guess of the default MTU size, 1300 bytes. 3x because that is the minimum bandwidth cost of notify / request / deliver.

A new ticket should be opened if more work is needed.

taokayan pushed a commit to taokayan/eos that referenced this issue May 15, 2019
Fix the logic in the buildkite docker tags
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant