remove unnecessary PoS checkpoint methods and variables (hyperledger#…

…6603) Signed-off-by: Gabriel Fukushima <gabrielfukushima@gmail.com>
kaleido-io · Jun 7, 2024 · a0072ef · a0072ef
1 parent 56bc852
commit a0072ef
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 31 deletions.
diff --git a/besu/src/main/java/org/hyperledger/besu/controller/MergeBesuControllerBuilder.java b/besu/src/main/java/org/hyperledger/besu/controller/MergeBesuControllerBuilder.java
@@ -198,7 +198,6 @@ protected MergeContext createConsensusContext(
                     .getTerminalTotalDifficulty()
                     .map(Difficulty::of)
                     .orElse(Difficulty.ZERO))
-            .setCheckpointPostMergeSync(syncConfig.isCheckpointPostMergeEnabled())
             .setPostMergeAtGenesis(isPostMergeAtGenesis);
 
     blockchain

diff --git a/consensus/merge/src/main/java/org/hyperledger/besu/consensus/merge/MergeContext.java b/consensus/merge/src/main/java/org/hyperledger/besu/consensus/merge/MergeContext.java
@@ -185,13 +185,6 @@ default boolean isChainPruningEnabled() {
     return false;
   }
 
-  /**
-   * Is checkpoint post merge sync.
-   *
-   * @return the boolean
-   */
-  boolean isCheckpointPostMergeSync();
-
   /**
    * Is configured for a post-merge from genesis.
    *

diff --git a/consensus/merge/src/main/java/org/hyperledger/besu/consensus/merge/PostMergeContext.java b/consensus/merge/src/main/java/org/hyperledger/besu/consensus/merge/PostMergeContext.java
@@ -70,7 +70,6 @@ public class PostMergeContext implements MergeContext {
   private final AtomicReference<Optional<BlockHeader>> terminalPoWBlock =
       new AtomicReference<>(Optional.empty());
   private final BlockValueCalculator blockValueCalculator = new BlockValueCalculator();
-  private boolean isCheckpointPostMergeSync;
   private boolean isPostMergeAtGenesis;
 
   // TODO: cleanup - isChainPruningEnabled will not be required after
@@ -92,7 +91,6 @@ public class PostMergeContext implements MergeContext {
   PostMergeContext(final Difficulty difficulty) {
     this.terminalTotalDifficulty = new AtomicReference<>(difficulty);
     this.syncState = new AtomicReference<>();
-    this.isCheckpointPostMergeSync = false;
   }
 
   /**
@@ -314,22 +312,6 @@ public boolean isChainPruningEnabled() {
     return isChainPruningEnabled;
   }
 
-  /**
-   * Sets checkpoint post merge sync.
-   *
-   * @param isCheckpointPostMergeSync the is checkpoint post merge sync
-   * @return the checkpoint post merge sync
-   */
-  public PostMergeContext setCheckpointPostMergeSync(final boolean isCheckpointPostMergeSync) {
-    this.isCheckpointPostMergeSync = isCheckpointPostMergeSync;
-    return this;
-  }
-
-  @Override
-  public boolean isCheckpointPostMergeSync() {
-    return this.isCheckpointPostMergeSync;
-  }
-
   @Override
   public boolean isPostMergeAtGenesis() {
     return this.isPostMergeAtGenesis;

diff --git a/consensus/merge/src/main/java/org/hyperledger/besu/consensus/merge/TransitionContext.java b/consensus/merge/src/main/java/org/hyperledger/besu/consensus/merge/TransitionContext.java
@@ -149,11 +149,6 @@ public Optional<BlockWithReceipts> retrieveBlockById(final PayloadIdentifier pay
     return postMergeContext.retrieveBlockById(payloadId);
   }
 
-  @Override
-  public boolean isCheckpointPostMergeSync() {
-    return false;
-  }
-
   @Override
   public boolean isPostMergeAtGenesis() {
     return postMergeContext.isPostMergeAtGenesis();