-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reduce ConnectionPool size used by send-transaction-service #33548
Conversation
size=2
[2023-10-04T21:35:55.736701590Z INFO solana_bench_tps::bench] Average TPS: 6890.304
[2023-10-04T22:42:48.290458451Z INFO solana_bench_tps::bench] Average TPS: 7473.9106
[2023-10-04T22:53:46.822453533Z INFO solana_bench_tps::bench] Average TPS: 7998.1733
[2023-10-04T23:00:28.413413709Z INFO solana_bench_tps::bench] Average TPS: 6373.1846
[2023-10-04T23:05:54.484348552Z INFO solana_bench_tps::bench] Average TPS: 6471.919 size=4
[2023-10-05T15:22:16.308344351Z INFO solana_bench_tps::bench] Average TPS: 7056.3384
[2023-10-05T15:26:18.136180982Z INFO solana_bench_tps::bench] Average TPS: 7965.6826
[2023-10-05T15:36:39.110740734Z INFO solana_bench_tps::bench] Average TPS: 8542.928
[2023-10-05T16:01:57.050631312Z INFO solana_bench_tps::bench] Average TPS: 7443.8564
[2023-10-05T17:20:10.456737141Z INFO solana_bench_tps::bench] Average TPS: 8129.8066 |
Just an FYI, when using tpu-client, the pool size still made big difference: size=2
[2023-10-04T19:50:29.126351162Z INFO solana_metrics::metrics] datapoint: bench-tps-lamport_balance balance=0i
[2023-10-04T19:51:55.739417025Z INFO solana_bench_tps::bench] Average TPS: 7004.1636
[2023-10-04T19:54:56.555236496Z INFO solana_bench_tps::bench] Average TPS: 6518.2456 size=4:
[2023-10-04T20:23:44.664669703Z INFO solana_bench_tps::bench] Average TPS: 10771.607
[2023-10-04T20:24:56.718196712Z INFO solana_bench_tps::bench] Average TPS: 10509.352
[2023-10-04T20:26:10.040882712Z INFO solana_bench_tps::bench] Average TPS: 10405.963 |
The send-transaction-service itself is not using tpu-client... Maybe we should consider utilizing that? @CriesofCarrots |
Codecov Report
@@ Coverage Diff @@
## master #33548 +/- ##
=========================================
- Coverage 81.9% 81.9% -0.1%
=========================================
Files 799 799
Lines 217339 217341 +2
=========================================
- Hits 178163 178105 -58
- Misses 39176 39236 +60 |
Problem
Reduce pool size for ConnectionCache used by send-transaction-service
Summary of Changes
Reduce pool size for ConnectionCache used by send-transaction-service to 2 from 4. No significant slow down of performance from bench-tps testing using rpc-client which is used by send-transaction-service. This will reduce active connections maintained both on the server and client. This will enable us to cache connections for more nodes.
Fixes #