Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

network-bridge-tx: batch peer reputation changes #7203

Closed
sandreim opened this issue May 9, 2023 · 0 comments · Fixed by #7214
Closed

network-bridge-tx: batch peer reputation changes #7203

sandreim opened this issue May 9, 2023 · 0 comments · Fixed by #7214
Assignees
@AndreiEres
Labels
I10-optimisation An enhancement to provide better overall performance in terms of time-to-completion for a task.

Comments

@sandreim
Copy link
Contributor

sandreim commented May 9, 2023

We could batch peer reputation changes such that we only send out a report message (which contains the aggregated changeset for all peers) to the network bridge once every 30 seconds for example. This would cost us more memory, but we would send out way less messages and avoid filling up the queue of the tx bridge.

Screenshot 2023-05-09 at 11 28 34
@sandreim sandreim added the I10-optimisation An enhancement to provide better overall performance in terms of time-to-completion for a task. label May 9, 2023
@sandreim sandreim moved this from Todo to In Progress in Road to 1k paravalidators and 200 parachains May 9, 2023
@AndreiEres AndreiEres mentioned this issue May 11, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@AndreiEres AndreiEres

Labels
I10-optimisation An enhancement to provide better overall performance in terms of time-to-completion for a task.
Projects
No open projects
Road to 1k paravalidators and 200 parachains
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Delay reputation updates paritytech/polkadot
2 participants
@AndreiEres @sandreim