Skip to content

Commit

Permalink
Complete the removal of minimumRequiredBlockNumber from constructors
Browse files Browse the repository at this point in the history
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
  • Loading branch information
fab-10 committed Jun 16, 2022
1 parent 4e96fae commit 8bdeadb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -253,11 +253,7 @@ public ChainDownloader createChainDownloader(final FastSyncState currentState) {

private CompletableFuture<FastSyncState> downloadPivotBlockHeader(final Hash hash) {
return RetryingGetHeaderFromPeerByHashTask.byHash(
protocolSchedule,
ethContext,
hash,
pivotBlockSelector.getMinRequiredBlockNumber(),
metricsSystem)
protocolSchedule, ethContext, hash, metricsSystem)
.getHeader()
.thenApply(
blockHeader -> {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ public class RetryingGetHeaderFromPeerByHashTask
final ProtocolSchedule protocolSchedule,
final EthContext ethContext,
final Hash referenceHash,
final long minimumRequiredBlockNumber,
final MetricsSystem metricsSystem) {
super(ethContext, 3, List::isEmpty, metricsSystem);
this.protocolSchedule = protocolSchedule;
Expand All @@ -55,10 +54,9 @@ public static RetryingGetHeaderFromPeerByHashTask byHash(
final ProtocolSchedule protocolSchedule,
final EthContext ethContext,
final Hash referenceHash,
final long minimumRequiredBlockNumber,
final MetricsSystem metricsSystem) {
return new RetryingGetHeaderFromPeerByHashTask(
protocolSchedule, ethContext, referenceHash, minimumRequiredBlockNumber, metricsSystem);
protocolSchedule, ethContext, referenceHash, metricsSystem);
}

@Override
Expand Down

0 comments on commit 8bdeadb

Please sign in to comment.