diff --git a/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/SyncConfig.java b/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/SyncConfig.java index 661cef6db03..2d31963d921 100644 --- a/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/SyncConfig.java +++ b/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/SyncConfig.java @@ -15,11 +15,10 @@ import static com.google.common.base.Preconditions.checkNotNull; +import java.util.Optional; import tech.pegasys.teku.infrastructure.unsigned.UInt64; import tech.pegasys.teku.networking.eth2.P2PConfig; -import java.util.Optional; - public class SyncConfig { public static final boolean DEFAULT_MULTI_PEER_SYNC_ENABLED = true; @@ -45,7 +44,7 @@ public class SyncConfig { private final int forwardSyncMaxPendingBatches; private final int forwardSyncMaxBlocksPerMinute; private final int forwardSyncMaxBlobSidecarsPerMinute; - private final Optional pinnedSyncSlot; + private final Optional pinnedSyncSlot; private SyncConfig( final boolean isEnabled, @@ -110,9 +109,9 @@ public int getForwardSyncMaxBlobSidecarsPerMinute() { return forwardSyncMaxBlobSidecarsPerMinute; } - public Optional getPinnedSyncSlot() { - return pinnedSyncSlot; - } + public Optional getPinnedSyncSlot() { + return pinnedSyncSlot; + } public static class Builder { private Boolean isEnabled; @@ -141,7 +140,7 @@ public SyncConfig build() { forwardSyncMaxPendingBatches, forwardSyncMaxBlocksPerMinute, forwardSyncMaxBlobSidecarsPerMinute, - pinnedSyncSlot); + pinnedSyncSlot); } private void initMissingDefaults() { @@ -188,7 +187,8 @@ public Builder forwardSyncMaxPendingBatches(final Integer forwardSyncMaxPendingB } public Builder pinnedSyncSlot(final Long pinnedSyncSlot) { - Optional.ofNullable(pinnedSyncSlot).ifPresent(slot -> this.pinnedSyncSlot = Optional.of(UInt64.valueOf(slot))); + Optional.ofNullable(pinnedSyncSlot) + .ifPresent(slot -> this.pinnedSyncSlot = Optional.of(UInt64.valueOf(slot))); return this; } diff --git a/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerCommonAncestorFinder.java b/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerCommonAncestorFinder.java index 0cd220945fd..01cb688894c 100644 --- a/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerCommonAncestorFinder.java +++ b/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerCommonAncestorFinder.java @@ -46,13 +46,12 @@ public class MultipeerCommonAncestorFinder { } public static MultipeerCommonAncestorFinder create( - final RecentChainData recentChainData, final EventThread eventThread, - final Optional pinnedSlot, final Spec spec) { + final RecentChainData recentChainData, + final EventThread eventThread, + final Optional pinnedSlot, + final Spec spec) { return new MultipeerCommonAncestorFinder( - recentChainData, - new CommonAncestor(recentChainData, pinnedSlot), - eventThread, - spec); + recentChainData, new CommonAncestor(recentChainData, pinnedSlot), eventThread, spec); } public SafeFuture findCommonAncestor(final TargetChain targetChain) { diff --git a/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerSyncService.java b/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerSyncService.java index b9545c1edb9..ae49176caff 100644 --- a/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerSyncService.java +++ b/beacon/sync/src/main/java/tech/pegasys/teku/beacon/sync/forward/multipeer/MultipeerSyncService.java @@ -13,6 +13,7 @@ package tech.pegasys.teku.beacon.sync.forward.multipeer; +import java.util.Optional; import org.hyperledger.besu.plugin.services.MetricsSystem; import tech.pegasys.teku.beacon.sync.events.SyncingStatus; import tech.pegasys.teku.beacon.sync.forward.ForwardSyncService; @@ -42,8 +43,6 @@ import tech.pegasys.teku.statetransition.util.PendingPool; import tech.pegasys.teku.storage.client.RecentChainData; -import java.util.Optional; - public class MultipeerSyncService extends Service implements ForwardSyncService { private final SyncStallDetector syncStallDetector; private final EventThread eventThread; @@ -102,7 +101,8 @@ public static MultipeerSyncService create( eventThread, blobSidecarManager, new PeerScoringConflictResolutionStrategy()), batchSize, maxPendingBatches, - MultipeerCommonAncestorFinder.create(recentChainData, eventThread, pinnedCommonAncestorSlot, spec), + MultipeerCommonAncestorFinder.create( + recentChainData, eventThread, pinnedCommonAncestorSlot, spec), timeProvider); final SyncController syncController = new SyncController( diff --git a/teku/src/main/java/tech/pegasys/teku/cli/options/P2POptions.java b/teku/src/main/java/tech/pegasys/teku/cli/options/P2POptions.java index eeede17d945..37e416ef694 100644 --- a/teku/src/main/java/tech/pegasys/teku/cli/options/P2POptions.java +++ b/teku/src/main/java/tech/pegasys/teku/cli/options/P2POptions.java @@ -223,12 +223,12 @@ The network interface(s) on which the node listens for P2P communication. private boolean multiPeerSyncEnabled = SyncConfig.DEFAULT_MULTI_PEER_SYNC_ENABLED; @Option( - names = {"--Xp2p-multipeer-sync-pinned-slot"}, - paramLabel = "", - showDefaultValue = Visibility.ALWAYS, - description = "Pin the sync to a specific slot", - hidden = true, - arity = "1") + names = {"--Xp2p-multipeer-sync-pinned-slot"}, + paramLabel = "", + showDefaultValue = Visibility.ALWAYS, + description = "Pin the sync to a specific slot", + hidden = true, + arity = "1") private Long pinnedSyncSlot; @Option( @@ -524,7 +524,7 @@ public void configure(final TekuConfiguration.Builder builder) { .forwardSyncMaxBlobSidecarsPerMinute(forwardSyncBlobSidecarsRateLimit) .forwardSyncBatchSize(forwardSyncBatchSize) .forwardSyncMaxPendingBatches(forwardSyncMaxPendingBatches) - .pinnedSyncSlot(pinnedSyncSlot)); + .pinnedSyncSlot(pinnedSyncSlot)); if (subscribeAllSubnetsEnabled) { builder