Skip to content

txpool: limit transactions outgoing messages (#8271)#8742

Merged
AskAlexSharov merged 2 commits intodevelfrom
pr/8271
Nov 20, 2023
Merged

txpool: limit transactions outgoing messages (#8271)#8742
AskAlexSharov merged 2 commits intodevelfrom
pr/8271

Conversation

@battlmonstr
Copy link
Contributor

@battlmonstr battlmonstr commented Nov 16, 2023

  • limit remote transactions re-broadcast to 3-6 peers
    Broadcasting to 100 peers generates too much outgoing traffic.
  • limit transactions count/size in PooledTransactions replies

Before it was sending 5.5-6.5 MiB/sec:

Screenshot 2023-11-17 at 15 50 15

With the fixes it stays at 3-3.5 MiB/sec:

Screenshot 2023-11-17 at 15 39 10

P.S. A baseline if everything is disabled (BroadcastPooledTxs, AnnouncePooledTxs, responses to GetPooledTransactions) is 0.5-1 MiB/sec.

@battlmonstr battlmonstr force-pushed the pr/8271 branch 4 times, most recently from 53d21ab to 8227e7a Compare November 16, 2023 11:20
@battlmonstr battlmonstr changed the title txpool: limit transactions broadcast to 5 peers (#8271) txpool: limit transactions broadcast to 3-5 peers (#8271) Nov 16, 2023
@battlmonstr battlmonstr changed the title txpool: limit transactions broadcast to 3-5 peers (#8271) txpool: limit transactions broadcast to 3-6 peers (#8271) Nov 16, 2023
@battlmonstr battlmonstr force-pushed the pr/8271 branch 3 times, most recently from f10d2ee to ff90106 Compare November 17, 2023 14:41
@battlmonstr battlmonstr changed the title txpool: limit transactions broadcast to 3-6 peers (#8271) txpool: limit transactions outgoing messages (#8271) Nov 17, 2023
@battlmonstr battlmonstr marked this pull request as ready for review November 17, 2023 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants