cerc-io/lighthouse
10
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Temporary Sync Work-Around (#1615)

Browse Source 
## Issue Addressed

#1590 

## Proposed Changes

This is a temporary workaround that prevents finalized chain sync from swapping chains. I'm merging this in now until the full solution is ready.
This commit is contained in:
Age Manning 2020-09-13 23:58:49 +00:00
parent c6abc56113
commit c9596fcf0e
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
beacon_node/network/src/sync/range_sync/chain_collection.rs
View File

@ -248,15 +248,17 @@ impl<T: BeaconChainTypes> ChainCollection<T> {
// Check if any chains become the new syncing chain // Check if any chains become the new syncing chain
if let Some(index) = self.finalized_syncing_index() { if let Some(index) = self.finalized_syncing_index() {
// There is a current finalized chain syncing // There is a current finalized chain syncing
let syncing_chain_peer_count = self.finalized_chains[index].peer_pool.len(); let _syncing_chain_peer_count = self.finalized_chains[index].peer_pool.len();
// search for a chain with more peers // search for a chain with more peers
if let Some((new_index, chain)) = if let Some((new_index, chain)) =
self.finalized_chains self.finalized_chains
.iter_mut() .iter_mut()
.enumerate() .enumerate()
.find(|(iter_index, chain)| { .find(|(_iter_index, _chain)| {
*iter_index != index && chain.peer_pool.len() > syncing_chain_peer_count false
// && *iter_index != index
// && chain.peer_pool.len() > syncing_chain_peer_count
}) })
{ {
// A chain has more peers. Swap the syncing chain // A chain has more peers. Swap the syncing chain