-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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 #