Correct bug in sync (#2615)

AgeManning · AgeManning · commit 00a7ef00362f · 2021-09-23T01:32:04.000Z
A bug that causes failed batches to continually download in a loop is corrected.
diff --git a/beacon_node/network/src/sync/range_sync/batch.rs b/beacon_node/network/src/sync/range_sync/batch.rs
@@ -257,7 +257,7 @@ impl<T: EthSpec, B: BatchConfig> BatchInfo<T, B> {
                     self.failed_download_attempts.push(peer);
                 }
                 self.state = if self.failed_download_attempts.len()
-                    >= B::max_batch_download_attempts as usize
+                    >= B::max_batch_download_attempts() as usize
                 {
                     BatchState::Failed
                 } else {

Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ impl<T: EthSpec, B: BatchConfig> BatchInfo<T, B> {`
`257`	`257`	`self.failed_download_attempts.push(peer);`
`258`	`258`	`}`
`259`	`259`	`self.state = if self.failed_download_attempts.len()`
`260`		`- >= B::max_batch_download_attempts as usize`
	`260`	`+ >= B::max_batch_download_attempts() as usize`
`261`	`261`	`{`
`262`	`262`	`BatchState::Failed`
`263`	`263`	`} else {`