search

tari-project / tari

The Tari protocol
https://tari.com
BSD 3-Clause "New" or "Revised" License
354 stars 219 forks source link

fix(core): reduce block messages with many connections #6602

Closed sdbondi closed 1 month ago

sdbondi commented 1 month ago

Description

fix(core): reduce block messages with many connections

Motivation and Context

When a seed node is connected to many hundreds of base node peers, it propagates blocks on all connections. This PR changes this to send to 20 random peers. Default propagation_factor is changed from 4 to 20.

How Has This Been Tested?

What process can a PR reviewer use to test or verify this change?

Breaking Changes

github-actions[bot] commented 1 month ago

Test Results (CI)

    3 files    129 suites   37m 26s :stopwatch: 1 325 tests 1 325 :white_check_mark: 0 :zzz: 0 :x: 3 973 runs  3 973 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 6054a998.

github-actions[bot] commented 1 month ago

Test Results (Integration tests)

36 tests  +36   36 :white_check_mark: +36   15m 21s :stopwatch: + 15m 21s 11 suites +11    0 :zzz: ± 0   2 files   + 2    0 :x: ± 0 

Results for commit 6054a998. ± Comparison against base commit 5bcddf44.