Tiny improvement: PeerManager and maximum discovery query (#3182)

## Issue Addressed

As [`Discovery` bounds the maximum discovery query](e88b18be09/beacon_node/lighthouse_network/src/discovery/mod.rs (L328)), `PeerManager` no need to handle it.

e88b18be09/beacon_node/lighthouse_network/src/discovery/mod.rs (L328)
This commit is contained in:
Akihito Nakano 2022-05-19 06:00:46 +00:00
parent 807283538f
commit 695f415590

View File

@ -841,21 +841,14 @@ impl<TSpec: EthSpec> PeerManager<TSpec> {
let outbound_only_peer_count = self.network_globals.connected_outbound_only_peers();
let wanted_peers = if peer_count < self.target_peers.saturating_sub(dialing_peers) {
// We need more peers in general.
// The maximum discovery query is for 16 peers, but we can search for less if
// needed.
std::cmp::min(
self.target_peers.saturating_sub(dialing_peers) - peer_count,
16,
)
// Note: The maximum discovery query is bounded by `Discovery`.
self.target_peers.saturating_sub(dialing_peers) - peer_count
} else if outbound_only_peer_count < self.min_outbound_only_peers()
&& peer_count < self.max_outbound_dialing_peers()
{
std::cmp::min(
self.max_outbound_dialing_peers()
.saturating_sub(dialing_peers)
- peer_count,
16,
)
self.max_outbound_dialing_peers()
.saturating_sub(dialing_peers)
- peer_count
} else {
0
};